Skip to content

Comments

Allow curly brackets within query/fragment#3

Open
knyjoh wants to merge 1 commit intoopis:masterfrom
knyjoh:allow-curly-brackets
Open

Allow curly brackets within query/fragment#3
knyjoh wants to merge 1 commit intoopis:masterfrom
knyjoh:allow-curly-brackets

Conversation

@knyjoh
Copy link

@knyjoh knyjoh commented Jun 12, 2025

A URL like this: https://example.com/?foo={bar} does not validate although it seems to be a valid URL.

Adding {} to the regex fixes the issue, which I believe is correct.

@sorinsarca
Copy link
Member

sorinsarca commented Jun 12, 2025

Accroding to RFC 3986 curly brackets must be percent-encoded.

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