HttpClientConnector sets credentials in constructor
---------------------------------------------------

                 Key: CMIS-21
                 URL: https://issues.apache.org/jira/browse/CMIS-21
             Project: Chemistry
          Issue Type: Bug
          Components: atompub, client
            Reporter: Ugo Cei
            Priority: Critical
         Attachments: patch.txt

The constructor for the HttpClientConnector takes a ContentManager as argument 
and sets up preemptive authentication using the CredentialsProvider stored in 
the ContentManager.

The problem is that, at this point, the CredentialsProvider is null as we 
haven't yet had a chance to call ContentManager.login, thus the 
CredentialsProvider will be null. The solution is to differ the setting up of 
preemptive auth to when ContentManager.login is called.

The attached patch does exactly this. It's a bit of a quick fix and uses a 
couple instanceof operations which I don't like much, so you might want to 
review it and make it better.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to