-
Notifications
You must be signed in to change notification settings - Fork 24
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
Respect Maven's proxy settings #398
Comments
Hi @chovyy, have you tried setting the MVN_OPTS environment variable for build system to the proxy variables? That looks like another viable approach - see this issue for examples: https://community.sonarsource.com/t/maven-plugin-not-using-proxy/43101. I don't have a corporate proxy that I can test a fix behind unfortunately. If the MVN_OPTS route doesn't work, then perhaps try updating the method AbstractDependencyTrackMojo.configureUnirest() to set the proxy globally for all HTTP calls from within the proxy and submitting a PR? |
As pointed out, the problem is not that I cannot set the proxy at all, but every plugin should respect the maven settings for such things. I will try to prepare a PR... Thanks for the hint. |
@pmckeown also tried to set a the proxy up in the Unirest config in AbstractDependencyTrackMojo but then get a lot of test failures with Unirest throwing an exception, for example:
|
the first part of my issue is now solved: #398 (comment) It was just that I didn't need the trailing /api on the base url |
From my workstation, our DT server is only reachable through a proxy.
For uploading BOMs, I have to set Java environment variables when calling this plugin:
That works fine, but it would be nice, if the plugin would take the proxy settings from Maven. In my
settings.xml
, I have something like:but this seems to be currently ignored by this plugin.
The text was updated successfully, but these errors were encountered: