Skip to content

feat: validate URL schemes in Anchor, IFrame and Page#open (CP: 24.9)#24939

Merged
platosha merged 1 commit into
24.9from
cherry/cherrypick-24924-to-24.9
Jul 8, 2026
Merged

feat: validate URL schemes in Anchor, IFrame and Page#open (CP: 24.9)#24939
platosha merged 1 commit into
24.9from
cherry/cherrypick-24924-to-24.9

Conversation

@mcollovati

Copy link
Copy Markdown
Collaborator

This PR cherry-picks changes from the original PRs #24539 and #24897 to branch 25.1, and also disables the validation by default.

The defaults when "safeUrlSchemes" configuration is missing is adjusted, so that URL validation is disabled by default, and all URLs are considered safe. This avoids unexpected behavior changes for existing users of Flow 25.1 and below.

The URL scheme validation feature could be enabled with configuration when necessary.

(cherry picked from commit 28e55a3)

…24897) (CP: 25.1) (#24924)

This PR cherry-picks changes from the original PRs #24539 and #24897 to
branch 25.1, and also disables the validation by default.

The defaults when "safeUrlSchemes" configuration is missing is adjusted,
so that URL validation is disabled by default, and all URLs are
considered safe. This avoids unexpected behavior changes for existing
users of Flow 25.1 and below.

The URL scheme validation feature could be enabled with configuration
when necessary.

(cherry picked from commit 28e55a3)
(cherry picked from commit 51048d1)
(cherry picked from commit 6eae250)
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

Test Results

1 295 files  ± 0  1 295 suites  ±0   1h 14m 19s ⏱️ -3s
8 952 tests +29  8 890 ✅ +29  62 💤 ±0  0 ❌ ±0 
9 264 runs   - 19  9 196 ✅  - 19  68 💤 ±0  0 ❌ ±0 

Results for commit 3320fea. ± Comparison against base commit 2380c51.

@platosha platosha merged commit 6e6b781 into 24.9 Jul 8, 2026
26 checks passed
@platosha platosha deleted the cherry/cherrypick-24924-to-24.9 branch July 8, 2026 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants