Skip to content

Update dev dependency selfsigned to v4, fix Node >=v24.5 compatibility, restore CI#54634

Closed
robhogan wants to merge 1 commit intomainfrom
export-D87643898
Closed

Update dev dependency selfsigned to v4, fix Node >=v24.5 compatibility, restore CI#54634
robhogan wants to merge 1 commit intomainfrom
export-D87643898

Conversation

@robhogan
Copy link
Copy Markdown
Contributor

Summary:
CI fails after attempting to update to Node v24 (latest LTS) because our dev-middleware dev-only dependency selfsigned uses a default key size incompatible with the version of OpenSSL in Node>=24.5. We see lots of:

error:0A00018F:SSL routines::ee key too small

This was mitigated in OSS CI by pinning our Node tests to before v24.5: #53013

This default is raised upstream in jfromaniello/selfsigned@bf687c8 , this bumps to v4 which includes that fix, and removes the temporary mitigation above.

Changelog:
[Internal]

Differential Revision: D87643898

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 21, 2025
@meta-codesync
Copy link
Copy Markdown

meta-codesync bot commented Nov 21, 2025

@robhogan has exported this pull request. If you are a Meta employee, you can view the originating Diff in D87643898.

…y, restore CI (#54634)

Summary:

CI fails after attempting to update to Node v24 (latest LTS) because our `dev-middleware` dev-only dependency `selfsigned` uses a default key size incompatible with the version of OpenSSL in Node>=24.5. We see lots of:

```
error:0A00018F:SSL routines::ee key too small
```

This was mitigated in OSS CI by pinning our Node tests to before v24.5: #53013

This default is raised upstream in jfromaniello/selfsigned@bf687c8 , this bumps to v4 which includes that fix, and removes the temporary mitigation above.

Changelog:
[Internal]

Reviewed By: cipolleschi

Differential Revision: D87643898
@meta-codesync meta-codesync bot closed this in f3f7e17 Nov 25, 2025
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Nov 25, 2025
@meta-codesync
Copy link
Copy Markdown

meta-codesync bot commented Nov 25, 2025

This pull request has been merged in f3f7e17.

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants