chemistry-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ugo Cei (JIRA)" <>
Subject [jira] Updated: (CMIS-21) HttpClientConnector sets credentials in constructor
Date Tue, 30 Jun 2009 16:56:47 GMT


Ugo Cei updated CMIS-21:

    Attachment:     (was: patch.txt)

> HttpClientConnector sets credentials in constructor
> ---------------------------------------------------
>                 Key: CMIS-21
>                 URL:
>             Project: Chemistry
>          Issue Type: Bug
>          Components: atompub, client
>            Reporter: Ugo Cei
>            Priority: Critical
> 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.

View raw message