Skip to content

Escape periods in URL regex#8

Open
PseudoResonance wants to merge 1 commit intodyatlov:masterfrom
PseudoResonance:master
Open

Escape periods in URL regex#8
PseudoResonance wants to merge 1 commit intodyatlov:masterfrom
PseudoResonance:master

Conversation

@PseudoResonance
Copy link

Periods in URLs need to be escaped to prevent the regex from matching it as any character.

Example: The Twitter provider generates the following regex ^https://[^/]*?.twitter.com/.*?/status/.*$, which works for Twitter subdomains, however it also matches things like fxtwitter.com.

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.

1 participant