-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
fix: respect https.cacert
option
#3820
Conversation
Let's add |
Codecov Report
@@ Coverage Diff @@
## master #3820 +/- ##
==========================================
- Coverage 92.80% 92.72% -0.09%
==========================================
Files 14 14
Lines 1321 1333 +12
Branches 463 470 +7
==========================================
+ Hits 1226 1236 +10
- Misses 87 89 +2
Partials 8 8
Continue to review full report at Codecov.
|
I will finish PR, here some more work |
@snitin315 Need documented after merge:
|
@alexander-akait I will take care of documentation 👍🏻 |
For Bugs and Features; did you add new tests?
No
Motivation / Use-Case
Fix #3814
Breaking Changes
None
Additional Info
https://nodejs.org/api/tls.html#tls_tls_createsecurecontext_options