-
Notifications
You must be signed in to change notification settings - Fork 747
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
Unable to setup java when using self-hosted runner behind proxy with authorization (must upgrade http-client version) #221
Comments
Bump http-client version to 1.0.11 to fix proxy auth error
Hello @pavelkrulec-ext96968. We released a new version of setup-java and updated v2 tag. Could you please confirm that it resolves your issue. |
Hello everyone. I'm closing the issue, because the new version of action was released. If you have any concerns feel free to ping us. |
Run actions/setup-java@v2 I am facing the same issue, even after using the V2 tag. |
Hello @kamlesh-singh-ey. Could you please try to use the latest major tag to test it ? |
Run actions/setup-java@v3 it is still the same for me, even on the proxy settings are picked up, because in the worker logs I can see |
Description:
TLDR: Update http-client dependency to 1.0.11.
We have a self-hosted runner which is behind a corporate proxy with authorization. We use CNTLM for the auth via NTLMv2 and the self-hosted runner is configured to use "http://localhost:3128" (where CNTLM listens) as HTTP and HTTPS proxy.
Once the Action is executed, it fails with following error:
This is due to a bug in http-client which has been fixed in version 1.0.11.
Task version:
v2.3.0
Platform:
Runner type:
Repro steps:
Expected behavior:
Action setup-java does not pass empty credentials to proxy, lets CNTLM use it's own configured credentials and downloads and installs java on the runner.
Actual behavior:
Action setup-java passes empty credentials to proxy, CNTLM forwards those empty credentials to parent proxy and gets 407 Proxy authorization required.
The text was updated successfully, but these errors were encountered: