#55623: fix: limit webRequest redirects across loader restarts to avoid crashes
Merged
Description of Change
Repeated webRequest redirects can grow the renderer's redirect history until it crashes. This change keeps each request's redirect count across loader restarts and rejects the 21st redirect with ERR_TOO_MANY_REDIRECTS.
Fixes #55622.
Checklist
- I have built and tested this change
- I have filled out the PR description
- I have reviewed and verified the changes
-
npm testpasses - tests are changed or added
- PR release notes describe the change in a way relevant to app developers, and are capitalized, punctuated, past tense, and no longer than 80 characters.
A few npm test cases fail in my local environment. The same failures also occur on the unmodified baseline.
Release Notes
Notes: Fixed renderer crashes caused by unbounded webRequest redirect loops.
Backports
43-x-y
PendingWaiting for a manual backport
Semver Impact
Major
Breaking changes
Minor
New features
Patch
Bug fixes
None
Docs, tests, etc.
Semantic Versioning helps users understand the impact of updates:
- Major (X.y.z): Breaking changes that may require code modifications
- Minor (x.Y.z): New features that maintain backward compatibility
- Patch (x.y.Z): Bug fixes that don't change the API
- None: Changes that don't affect using facing parts of Electron