Skip to content
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

Do not allow username,password,port for URLs without a host or file URLs #411

Merged
merged 1 commit into from
Nov 6, 2017

Conversation

valenting
Copy link
Collaborator

@valenting valenting commented Nov 2, 2017

Imports tests from web-platform-tests/wpt@0e6a90f and tracks URL spec change at whatwg/url#224


This change is Reviewable

@SimonSapin
Copy link
Member

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit f9d7971 has been approved by SimonSapin

@bors-servo
Copy link
Contributor

⌛ Testing commit f9d7971 with merge 211bf4a...

bors-servo pushed a commit that referenced this pull request Nov 6, 2017
Do not allow username,password,port for URLs without a host or file URLs

Imports tests from web-platform-tests/wpt@0e6a90f and tracks URL spec change at whatwg/url#224

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-url/411)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - status-travis
Approved by: SimonSapin
Pushing 211bf4a to master...

@bors-servo bors-servo merged commit f9d7971 into servo:master Nov 6, 2017
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.

3 participants