Skip to content
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

#972: add trustInsecureDownloadRoot parameter to plugins #1027

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sdoeringNew
Copy link

Summary

Different download roots for Node, NPM, Yarn, etc. can be specified.
Those download roots might have HTTPS connections with self signed certificate.
For all those connections an additional plugin configuration has been added.
It's called: trustInsecureDownloadRoot
This might solve issue #972. The issue description is more or less vague.
For me it fixed the local problem with self signed certificates.

Tests and Documentation

Of course, tests and documentation have been added.
And while adding tests a bug was discovered. Downloads via Proxy with BasicAuth did not work.
This has been fixed while at it.

While adding unit tests by default, a bug was discovered when downloading via proxy with BasicAuth. This has also been fixed.
@west1cles
Copy link

@sdoeringNew thank you so much for this. I was banging my head on my desk for days trying to bypass some cert issues and I was able to get passed it with this.

@sdoeringNew
Copy link
Author

Glad to here it. 👍
Unfortunately the PR is still not approved.

@wajda
Copy link

wajda commented Jun 23, 2023

Hi, is there any progress on this? Thank you.

@Shrishti217
Copy link

@sdoeringNew thank you so much for this. I was banging my head on my desk for days trying to bypass some cert issues and I was able to get passed it with this.

-->Can you please let me know how did you resolve this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants