[GitHub] ant-ivy pull request #64: IVY-1280 Support preemptive authentication

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] ant-ivy pull request #64: IVY-1280 Support preemptive authentication

bodewig
GitHub user twogee opened a pull request:

    https://github.com/apache/ant-ivy/pull/64

    IVY-1280 Support preemptive authentication

    I adopted a patch by @AurelienLourot

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/twogee/ant-ivy ivy-1280

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/ant-ivy/pull/64.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #64
   
----
commit 2422f51ffad184465a5a9bacdc92e44f7f4a756a
Author: twogee <g.grigelionis@...>
Date:   2018-02-01T10:40:10Z

    IVY-1280 Support preemptive authentication

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant-ivy issue #64: IVY-1280 Support preemptive authentication

bodewig
Github user AurelienLourot commented on the issue:

    https://github.com/apache/ant-ivy/pull/64
 
    Note that I didn't push it upstream because of this open point: https://github.com/tt-gf/ant-ivy/pull/2#discussion_r135625987
   
    So this might break some authentication schemes, I'm not really sure.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant-ivy issue #64: IVY-1280 Support preemptive authentication

bodewig
In reply to this post by bodewig
Github user twogee commented on the issue:

    https://github.com/apache/ant-ivy/pull/64
 
    I have noticed that. Perhaps it's enough to document that preemptive authentication only works with basic scheme.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant-ivy issue #64: IVY-1280 Support preemptive authentication

bodewig
In reply to this post by bodewig
Github user twogee commented on the issue:

    https://github.com/apache/ant-ivy/pull/64
 
    Looking at [examples](https://github.com/apache/httpcomponents-client/blob/4.6.x/httpclient/src/examples/org/apache/http/examples/client/ClientPreemptiveDigestAuthentication.java), digest authentication uses the same cache, only sets a different scheme.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant-ivy issue #64: IVY-1280 Support preemptive authentication

bodewig
In reply to this post by bodewig
Github user twogee commented on the issue:

    https://github.com/apache/ant-ivy/pull/64
 
    I would like to add the preemptive authentication flag to the newly minted `TimeoutConstrainedURLHandler` to avoid BWC issues.  Any objections, @jaikiran ?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant-ivy issue #64: IVY-1280 Support preemptive authentication

bodewig
In reply to this post by bodewig
Github user twogee commented on the issue:

    https://github.com/apache/ant-ivy/pull/64
 
    @AurelienLourot we moved the discussion to [JIRA](https://issues.apache.org/jira/browse/IVY-1280)


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]