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

Test errors with Node.js 17 #1061

Closed
targos opened this issue Oct 14, 2021 · 5 comments · Fixed by #1063
Closed

Test errors with Node.js 17 #1061

targos opened this issue Oct 14, 2021 · 5 comments · Fixed by #1063
Labels
bug Something isn't working

Comments

@targos
Copy link
Member

targos commented Oct 14, 2021

See CITGM result: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/2778/nodes=fedora-latest-x64/testReport/junit/(root)/citgm/undici_v4_7_2/

@targos targos added the bug Something isn't working label Oct 14, 2021
@mcollina
Copy link
Member

This is due to nodejs/node#39987 and it's ok.

@targos
Copy link
Member Author

targos commented Oct 15, 2021

Some are also due to the OpenSSL update:

 # Subtest: Client using valid client certificate
         1..5
         not ok 1 - error:0A00018E:SSL routines::ca md too weak
           ---
           origin:
             at:
               line: 65
               column: 13
               file: node:internal/tls/secure-context

mcollina added a commit that referenced this issue Oct 15, 2021
@mcollina
Copy link
Member

I cannot reproduce that locally using -rc.1.

@mcollina
Copy link
Member

The OpenSSL error looks suspicious and to be investigated. It does not reproduce on the two machines I have available (Mac and Linux)

@mcollina
Copy link
Member

I'm going to release the dns fixes as a patch, let us know if the OpenSSL persist.

@mcollina mcollina reopened this Nov 5, 2021
KhafraDev pushed a commit to KhafraDev/undici that referenced this issue Jun 23, 2022
@Trott Trott closed this as completed in 14ded8b Nov 26, 2022
crysmags pushed a commit to crysmags/undici that referenced this issue Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants