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

Elliotte Rusty Harold updated WAGON-396:
----------------------------------------
    Description: 
I recently implemented SPNEGO authentication for the wagon-http provider. This 
implementation required a extension of DefaultHttpClient. So far this support 
lives in a separate modules. Unfortunately overriding the client initialization 
is not easily possible in the current code and requires nearly full duplication 
of wagon-http code.

I would suggest to move the instantiation of the client from 
AbstractHttpClientWagon.openConnectionInternal() into a separate factory 
method, e.g.

{code}
 protected DefaultHttpClient createHttpClient( ClientConnectionManager 
connManager ) {
        return new DefaultHttpClient(connManager);
    }

    public void openConnectionInternal()
    {
        repository.setUrl( getURL( repository ) );
        client = this.createHttpClient( getConnectionManager() );
{code}

In case you are interested to integrate the SPNEGO support, I will be happy to 
share.

  was:
I recently implemented SPNEGO authentication for the wagon-http provider. This 
implementation required a extension of DefaultHttpClient. So far this support 
lives in a separate modules. Unfortunately overriding the client initialization 
is not easily possible in the current code and requires nearly full duplication 
of wagon-http code.

I would suggest to move the instantiation of the client from 
AbstractHttpClientWagon.openConnectionInternal() into a seperate factory 
method, e.g.

{code}
 protected DefaultHttpClient createHttpClient( ClientConnectionManager 
connManager ) {
        return new DefaultHttpClient(connManager);
    }

    public void openConnectionInternal()
    {
        repository.setUrl( getURL( repository ) );
        client = this.createHttpClient( getConnectionManager() );
{code}

In case you are interested to integrate the SPNEGO support, I will be happy to 
share.


> AbstractHttpClientWagon extensibility (SPNEGO Authentication)
> -------------------------------------------------------------
>
>                 Key: WAGON-396
>                 URL: https://issues.apache.org/jira/browse/WAGON-396
>             Project: Maven Wagon
>          Issue Type: Improvement
>          Components: wagon-http
>    Affects Versions: 2.5
>            Reporter: Moritz Bechler
>            Priority: Minor
>
> I recently implemented SPNEGO authentication for the wagon-http provider. 
> This implementation required a extension of DefaultHttpClient. So far this 
> support lives in a separate modules. Unfortunately overriding the client 
> initialization is not easily possible in the current code and requires nearly 
> full duplication of wagon-http code.
> I would suggest to move the instantiation of the client from 
> AbstractHttpClientWagon.openConnectionInternal() into a separate factory 
> method, e.g.
> {code}
>  protected DefaultHttpClient createHttpClient( ClientConnectionManager 
> connManager ) {
>       return new DefaultHttpClient(connManager);
>     }
>     public void openConnectionInternal()
>     {
>         repository.setUrl( getURL( repository ) );
>         client = this.createHttpClient( getConnectionManager() );
> {code}
> In case you are interested to integrate the SPNEGO support, I will be happy 
> to share.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to