-
Notifications
You must be signed in to change notification settings - Fork 161
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
SSL Error After Upgrading Application To Node 18 #239
Comments
We hit this issue too, this is not a node or node-vault issue. The latest version of OpenSSL, which apparently the latest node container is using requires the destination to not have unsafe renegotiation. There are workarounds (configurations you can put into the openssl config), or you can get the target web service to secure itself properly. |
@tcs-cclaflin - heya! sorry you bump into that; I'd follow @TJM 's recommendation on upgrading the other server as it makes less sense to unsecure this app in order for it to be backward compatible; Please feel free to open a PR to this bug once finding the appropriate location for these configurations |
Thank you for the replies, unfortunately we don't control the vault endpoint so will have to coordinate with multiple other teams (infrastructure, security, etc.) to try and get it changed. seeing this in
openssl isn't even installed in the image so unsure if creating a custom openssl config would work and was hoping to not have to go that route. |
Any updates on how to resolve this issue ? |
We have an application that connects to vault that we are trying to upgrade from Node 14 to Node 18. Node 18 introduced a change in the ssl library that is causes SSL issues with our instance of vault. The error we are seeing is:
Recommended solutions when searching that error is to change the SSL configuration at the endpoint (which we can't do easily at this time) or implement
SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION
orSSL_OP_LEGACY_SERVER_CONNECT
as described here: nodejs/node#45378I found an example of adding securityOptions in pass_request_options.js and tried adding multiple variations (secureOptions, securityOptions, agentOptions, etc.) of that to the options section in the application:
Can someone confirm what the correct option should be for this package and if this is the correct location to try and implement it?
The text was updated successfully, but these errors were encountered: