Skip to content

Fix Electron URL leaking in envelope via missing electron flag#1043

Closed
Copilot wants to merge 3 commits intoninosa/UrlUpdatefrom
copilot/sub-pr-970
Closed

Fix Electron URL leaking in envelope via missing electron flag#1043
Copilot wants to merge 3 commits intoninosa/UrlUpdatefrom
copilot/sub-pr-970

Conversation

Copy link
Contributor

Copilot AI commented Feb 25, 2026

  • Revert !!metadata.electron change in envelope.ts — the default false value for the electron flag in scrub.url() is the intended behavior

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…ocal filesystem paths

Co-authored-by: Niyibitanga <83887079+Niyibitanga@users.noreply.github.com>
Copilot AI changed the title [WIP] Update default Electron flag value in URL truncation PR Fix Electron URL leaking in envelope via missing electron flag Feb 25, 2026
Copilot AI requested a review from Niyibitanga February 25, 2026 00:05
Copy link
Contributor

@Niyibitanga Niyibitanga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like I said, the changes are unnecessary because the default value for the Electron flag is false and that's the intended behavior

…d behavior

Co-authored-by: Niyibitanga <83887079+Niyibitanga@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 25, 2026

Like I said, the changes are unnecessary because the default value for the Electron flag is false and that's the intended behavior

Reverted in c479e52. envelope.ts is back to calling scrub.url(location.href) without explicitly passing the electron flag.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants