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

fix: Node.js atob input validation #35415

Merged
merged 1 commit into from Aug 25, 2022
Merged

fix: Node.js atob input validation #35415

merged 1 commit into from Aug 25, 2022

Conversation

codebytere
Copy link
Member

Description of Change

Closes #31018.

Cherry-picks

to fix inconsistencies between Node.js' implementation of atob and the WHATWG spec.

Checklist

Release Notes

Notes: Fixed a bug where calling atob in the renderer process could fail under some circumstances.

@codebytere codebytere added semver/patch backwards-compatible bug fixes target/19-x-y target/21-x-y PR should also be added to the "21-x-y" branch. labels Aug 23, 2022
@codebytere codebytere requested review from a team as code owners August 23, 2022 09:17
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Aug 23, 2022
@codebytere codebytere changed the title fix: Node.js atob input validation fix: Node.js atob input validation Aug 23, 2022
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Aug 24, 2022
@zcbenz zcbenz merged commit 22ff2b6 into main Aug 25, 2022
@zcbenz zcbenz deleted the fix-atob-mismatch branch August 25, 2022 06:55
@release-clerk
Copy link

release-clerk bot commented Aug 25, 2022

Release Notes Persisted

Fixed a bug where calling atob in the renderer process could fail under some circumstances.

@trop
Copy link
Contributor

trop bot commented Aug 25, 2022

I have automatically backported this PR to "21-x-y", please check out #35443

@trop trop bot removed the target/21-x-y PR should also be added to the "21-x-y" branch. label Aug 25, 2022
@trop
Copy link
Contributor

trop bot commented Aug 25, 2022

I have automatically backported this PR to "19-x-y", please check out #35444

@trop
Copy link
Contributor

trop bot commented Aug 25, 2022

I have automatically backported this PR to "20-x-y", please check out #35445

@trop trop bot added in-flight/19-x-y merged/21-x-y PR was merged to the "21-x-y" branch. and removed target/19-x-y labels Aug 25, 2022
khalwa pushed a commit to solarwindscloud/electron that referenced this pull request Feb 22, 2023
fix: Node.js atob input validation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged/21-x-y PR was merged to the "21-x-y" branch. semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: ContextIsolation changes page behavior and breaks protonmail login
3 participants