Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Proxy Authentication kwargs not used #23

Merged
merged 1 commit into from

2 participants

@juergenhoetzel

Proxy.setAuthentication returns a new proxy object, so the auth
configuration was never used.

@juergenhoetzel juergenhoetzel Fix proxy auth configuration
Proxy.setAuthentication returns a new proxy object, so the auth
configuration was never used.
b8eff56
@cemerick cemerick merged commit b8eff56 into cemerick:master
@cemerick
Owner

Thank you! :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Apr 5, 2012
  1. @juergenhoetzel

    Fix proxy auth configuration

    juergenhoetzel authored
    Proxy.setAuthentication returns a new proxy object, so the auth
    configuration was never used.
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 3 deletions.
  1. +2 −3 src/main/clojure/cemerick/pomegranate/aether.clj
View
5 src/main/clojure/cemerick/pomegranate/aether.clj
@@ -160,9 +160,8 @@
:as proxy} ]
(if (and repo host port)
(let [prx-sel (doto (DefaultProxySelector.)
- (.add (doto (Proxy. type host port nil)
- (set-authentication proxy))
- non-proxy-hosts))
+ (.add (set-authentication (Proxy. type host port nil) proxy)
+ non-proxy-hosts))
prx (.getProxy prx-sel repo)]
(.setProxy repo prx))
repo))
Something went wrong with that request. Please try again.