-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
test: fix test-tls-client-auth test for OpenSSL32 #54610
Conversation
Refs: nodejs#53382 Refs: nodejs#53384 Same change as in 53384 where OpenSSL32 returns a slightly different error but for a different test. Signed-off-by: Michael Dawson <midawson@redhat.com>
Run on OpenSSL32 - https://ci.nodejs.org/job/richardlau-node-test-commit-linux-containered/33/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment is an observation and non-blocking.
Looks like test is still failing, this time due to key size: https://ci.nodejs.org/job/richardlau-node-test-commit-linux-containered/33/nodes=ubuntu2204_sharedlibs_openssl32_x64/testReport/junit/(root)/parallel/test_tls_client_auth/ |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #54610 +/- ##
==========================================
- Coverage 87.34% 87.33% -0.01%
==========================================
Files 649 649
Lines 182562 182562
Branches 35023 35029 +6
==========================================
- Hits 159451 159446 -5
+ Misses 16396 16383 -13
- Partials 6715 6733 +18 |
@richardlau I'm hoping that the CA key too small will be fixed by my other PR so this one should still land. |
Co-authored-by: Richard Lau <rlau@redhat.com>
Ok. It'll need a clean CI, so we land the other one and then rebase this on top. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Landed in e9cd476 |
Refs: nodejs#53382 Refs: nodejs#53384 Same change as in 53384 where OpenSSL32 returns a slightly different error but for a different test. Signed-off-by: Michael Dawson <midawson@redhat.com> PR-URL: nodejs#54610 Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Refs: #53382
Refs: #53384
Same change as in 53384 where OpenSSL32 returns a slightly different error but for a different test.