New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Correctly parse all URL schemes in UrlHelper::isLookLikeUrl(). #8744
Conversation
There are two integration tests failing now. Maybe we should check where the |
…g + fix SitesManager/ApiTest.
Fixed the failing tests. Will fix other failures if they show up.
We should probably replace this w/ a utility library in 3.0. For LTS, big changes like this would be counterproductive. |
Failing UI test is a random failure |
page.evaluate(function () { | ||
// cannot use sendKeys since quickform does not use placeholder attribute |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@diosmosis curious why it's still above page.sendKeys('input[name=siteName]', 'Serenity');
if we cannot use sendKeys?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Forgot to delete it :)
LGTM and works! |
Correctly parse all URL schemes in UrlHelper::isLookLikeUrl().
As title. Changed the regex to match all allowed schemes (as per the RFC: https://tools.ietf.org/html/rfc3986#section-3.1).
Fixes #8722