Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
bug #20102 [Validator] Url validator not validating hosts ending in a…
… number (gwkunze) This PR was squashed before being merged into the 2.7 branch (closes #20102). Discussion ---------- [Validator] Url validator not validating hosts ending in a number | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | yes | New feature? | no | BC breaks? | no (Should validate slightly more urls) | Deprecations? | no | Tests pass? | yes | Fixed tickets | None that I could find | License | MIT | Doc PR | We ran into an issue where the URL validator wasn't validating URL that we got from a legitimate (intranet) URL. Afaict from skimming RFCs the URL `http://myhost123/` is valid, however the current regular expression doesn't accept the last part of a hostname (in this case the entire hostname is the last part as there are no periods in the host). I've fixed this by adding the numbers unicode character property to the regular expression, I'm not sure but symbol characters (`\pS`) might need to be added to the group as well. Commits ------- 934c434 [Validator] Url validator not validating hosts ending in a number
- Loading branch information