TRIVIAL: clean up httpclient logging #786

Merged
merged 1 commit into from Apr 3, 2013

Projects

None yet

4 participants

@jdillon

No description provided.

@cstamas

+1

@ifedorenko ifedorenko commented on the diff Mar 25, 2013
.../sonatype/nexus/apachehttpclient/Hc4ProviderBase.java
@@ -181,7 +171,7 @@ protected void configureAuthentication( final DefaultHttpClient httpClient,
authScope = proxyHost.toHostString() + " proxy";
}
- List<String> authorisationPreference = new ArrayList<String>( 2 );
+ List<String> authorisationPreference = Lists.newArrayListWithExpectedSize(3);
@ifedorenko
ifedorenko Mar 25, 2013

Logging cleanup?

@jdillon
jdillon Mar 25, 2013

Obviously not :-P I did notice when looking over the logging here that when ntlm was configured that the size is expected to be 3 not 2. Its a minor and safe optimization which I didn't bother to track in a separate change.

@jdillon jdillon merged commit 0ebd3ed into master Apr 3, 2013
@jdillon jdillon deleted the httpclient-logging-tidy branch Apr 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment