Hi, will this be fixed? The above link does not include the complete Thread by the way. Here is the correct link:

http://mail-archives.apache.org/mod_mbox/maven-users/200510.mbox/%3C9e3862d80510171832u1e4204ds81b7c9ae3e5b9da7@mail.gmail.com%3E

Problem is that the proxy actually gets set as system property regardless of the active flag. this also causes problems for other libraries that look at this property when you for instance run svnkit inside an application in the jetty-maven-plugin. See: http://jira.codehaus.org/browse/JETTY-694

This patch produces the intended behavior:

• empty proxy section --> no active proxy
• only inactive proxies --> no active proxy
• at least one active proxy --> first active proxy is returned

Test cases included.

I added an integration test as well. It'd be great if you could also submit those in future

Thanks for the patch!

Yup, I'll do that next time.

Actually I just submitted my very first IT (see MNG-3641): kudos to Benjamin for teaching me the "tricks of the trade"! So ... no more excuses for me!

Merged to 3.x in r759552.