-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for maven proxy tag <active> #1074
Comments
I'll look into implementing this. Thanks for the suggestion. |
Great! Thanks. I updated the original report as the tag was not showing (markdown issue) and my grammer was off a bit. |
I have a related problem. When I have two active proxy definitions (one for http and another for https) in my settings.xml I get this error:
The workaround is to disable the http proxy while I do the dependency check. |
One approach to solve this is to use a class like The idea is that the list of active proxies is encapsulated by |
@davidmoten you are aware that more than one active proxy is not supported by maven? And that the protocol of the settings refers to the protocol of the proxy-server and not of the URL that is accessed? Settings reference |
@jeremylong Shall I take a look into this and prepare a PR? Or do you already have local coding in progress for it? |
@aikebah nope, wasn't aware, thanks for the heads-up and the links! |
Already solved with the fixes of #831 / #1493 in 19ce3f4 and 8190a86 @jeremylong this enhancement request should be closed. Was fixed with version 3.3.4 |
Maven has a tag called
<active>
in the proxies of settings.xmlhttps://maven.apache.org/guides/mini/guide-proxies.html
Currently this plugin does not handle the active tag and instead throws an error that the mavenSettingsProxyId must be set.
It would be a nice improvement if it handled that tag so we didn't have to put proxy information into our pom.
In my case, I have two proxies listed with one active 'true' and one active 'false'. This is not an issue if I remove one of those proxies regardless of the active setting.
The text was updated successfully, but these errors were encountered: