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

Update request.ts for SELF_SIGNED_CERT_IN_CHAIN node error #55

Closed
wants to merge 1 commit into from
Closed

Update request.ts for SELF_SIGNED_CERT_IN_CHAIN node error #55

wants to merge 1 commit into from

Conversation

PeterMourfield
Copy link

set rejectUnauthorized: false so that the Azure DevOps scanner can communicate with an instance of SonarQube that uses self-signed certificates. This avoids the node error SELF_SIGNED_CERT_IN_CHAIN

set rejectUnauthorized: false so that the Azure DevOps scanner can communicate with an instance of SonarQube that uses self-signed certificates. This avoids the node error SELF_SIGNED_CERT_IN_CHAIN
@PeterMourfield
Copy link
Author

Looking back I believe this should be set via a sonar.* property that users could disable based on their security posture, defaulted to true obviously.

@ghost
Copy link

ghost commented Sep 17, 2019

Thanks for submitting this PR. As there is no traction on this for a long time, i'm closing it. If this is still a feature valuable for you, please consider posting in our community forum a feature request.

@ghost ghost closed this Sep 17, 2019
@NorDroN
Copy link

NorDroN commented Oct 22, 2019

Hi!
I have the same problem.
Why this PR was rejected?
Community forum

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants