[jira] Assigned: (CMIS-21) HttpClientConnector sets credentials in constructor

["Florent Guillaume (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/CMIS-21?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Florent Guillaume reassigned CMIS-21:
-------------------------------------

    Assignee: Florent Guillaume

> 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
>            Assignee: Florent Guillaume
>            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.