Github user semancik commented on the issue:

    https://github.com/apache/httpclient/pull/66
  
    Another note: if you remove trace devel then there will be no way how to 
debug the code. I think I have made that quite clear in the comments. The first 
thing to know about CredSSP is that it is usually TLS-in-TLS protocol and there 
is no way how to diagnose the issues using a packet sniffer or any similar 
tool. The devel trace is only real option if you want to have the 
implementation maintainable. I suggest you have a look at MS-CSSP and not just 
consider the NTLM part in isolation. In fact it is not easy to enable the devel 
trace by mistake. So what is the point? You are all the rage about how 
formatting is an obstacle to maintenance. And that can be easily fixed. And 
then you are suggesting to remove one of the things that make maintainability 
possible? I'm sorry, but I do not really get it.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to