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

NodeJS bundled SSL root CA cert for AddTrust expired 30th May 2020 (breaks VSTS agent for us) #2993

Closed
wallyhall opened this issue Jun 1, 2020 · 7 comments

Comments

@wallyhall
Copy link

I'm raising this issue to make you aware of an issue I've raised with NodeJS - as it affects Azure DevOps heavily (for us, and I expect other users too).

Since the 30th May 2020 we've had several VSTS/DevOps agent plugins break - usually reporting errors to do with SSL certificates having expired.

If I understand correctly, the Azure DevOps agent installs a bundled NodeJS runtime - which is heavily used by both the agent itself and all installed plugins (like, notably in our case, the SonarQube plugin - which I've raised an issue on too).

NodeJS appears to be shipping an expired CA root certificate for AddTrust based SSL certificates - causing all our build pipelines to start failing (due to our reliance on connectivity with services secured by such SSL certs).

Hoping this can be fixed quickly by upstream NodeJS - and then pushed by Microsoft with an updated DevOps agent?

@wallyhall
Copy link
Author

This has been fixed (subject to proving I guess) up-stream at NodeJS.

They also have provided details on building NodeJS with an externally provided CA root.

@LucaBlackDragon
Copy link

Temporary and quite insecure fix:
define a variable NODE_TLS_REJECT_UNAUTHORIZED with value 0 in the pipeline in order to disable Node.js checks on certificates validity

@github-actions
Copy link

github-actions bot commented Dec 6, 2020

This issue has had no activity in 180 days. Please comment if it is not actually stale

@mjroghelia
Copy link
Contributor

@anatolybolshakov Could you please get someone to pull look into this and determine whether the upstream Node fix has backported to the version of Node 10 that is currently rolling out with agent 2.179.0?

@mjroghelia mjroghelia removed the stale label Dec 14, 2020
@mjroghelia mjroghelia reopened this Dec 14, 2020
@LucaBlackDragon
Copy link

LucaBlackDragon commented Dec 18, 2020

@anatolybolshakov Could you please get someone to pull look into this and determine whether the upstream Node fix has backported to the version of Node 10 that is currently rolling out with agent 2.179.0?

@mjroghelia I don't think so:

But I can confirm that things are working for us with Agent v2.153.2 so...

¯\_(ツ)_/¯

@anatolybolshakov
Copy link
Contributor

Hi @LucaBlackDragon yes, you're right - it doesn't seem to be backported to v10.23.0 (according to the discussion also)
@wallyhall do you still face with this issue?
cc @mjroghelia

@github-actions
Copy link

This issue has had no activity in 180 days. Please comment if it is not actually stale

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

No branches or pull requests

4 participants