{{ message }}
Repository navigation
chore: rework BoringSSL with Node.js incompatibilities patch - #55039
Merged
Merged
Conversation
* chore: adopt nodejs/node#66752 in BoringSSL node patch Rework fix_handle_boringssl_and_openssl_incompatibilities.patch to carry the OPENSSL_SIZE_T changes from nodejs/node#66752 verbatim instead of Electron's own size_t edits, so the patch drops out cleanly once Electron updates to a Node.js version that includes it. Also drop the `#if 0` guards in src/node_metadata.h and src/node_options.h, restoring upstream's `#if HAVE_OPENSSL`. BoringSSL provides the OpenSSL headers they include and the quic guard keeps openssl/quic.h out, so the guards are no longer needed. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XoTevKUU5Yqx5KULydnHjv * fix: add BoringSSL include path to node snapshot targets With the `#if 0` guards gone, src/node_options.h and src/node_metadata.h include <openssl/opensslv.h> and <openssl/crypto.h> again whenever HAVE_OPENSSL is set. The :node_snapshot and :node_snapshot_stub source sets get HAVE_OPENSSL=1 from :node_internal_config but did not depend on //third_party/boringssl, so they had no BoringSSL include path and the Windows build failed with "'openssl/opensslv.h' file not found" when compiling node_snapshot.cc. Depend on //third_party/boringssl from both targets when node_use_openssl is set, so they see the same headers as libnode. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XoTevKUU5Yqx5KULydnHjv * chore: use $node_openssl_path for node snapshot OpenSSL dep Depend on "$node_openssl_path" rather than hard-coding //third_party/boringssl in the :node_snapshot and :node_snapshot_stub targets, matching how upstream's unofficial.gni references OpenSSL. Electron sets node_openssl_path = "//third_party/boringssl" in build/args/all.gn, so the resolved dependency is unchanged. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XoTevKUU5Yqx5KULydnHjv --------- Co-authored-by: Claude <noreply@anthropic.com> (cherry picked from commit deec352) Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FEtfbdwkKabsmYWAF8Sqsw
4 tasks done
dsanders11
approved these changes
Oct 16, 2026
MarshallOfSound
approved these changes
Oct 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Requested by David Babenko-Sanders · Slack thread
Description of Change
Manually backport #55037. See that PR for details.
CC @dsanders11
The cherry-pick conflicted only in the generated index and hunk headers of the node patches, because this branch's node patch series differs from main's (src_backport_the_perfetto_trace_agent.patch on main is src_add_perfetto_trace_agent.patch here), so I rebuilt them by applying this branch's series plus the change to Node.js v24.22.0 and re-exporting with script/git-export-patches, which gives the same Node.js source delta as on main.
Checklist
npm testpasses🤖 Generated with Claude Code
https://claude.ai/code/session_01FEtfbdwkKabsmYWAF8Sqsw
Release Notes
Notes: none
Generated by Claude Code