Skip to content

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

Merged
bors-servo merged 1 commit into
servo:masterfrom
valenting:no-host-username
Nov 6, 2017
Merged

Do not allow username,password,port for URLs without a host or file URLs#411
bors-servo merged 1 commit into
servo:masterfrom
valenting:no-host-username

Conversation

@valenting

@valenting valenting commented Nov 2, 2017

Copy link
Copy Markdown
Collaborator

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


This change is Reviewable

@SimonSapin

Copy link
Copy Markdown
Member

@bors-servo r+

@bors-servo

Copy link
Copy Markdown
Contributor

📌 Commit f9d7971 has been approved by SimonSapin

@bors-servo

Copy link
Copy Markdown
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
Copy Markdown
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
takumi-earth pushed a commit to earthlings-dev/rust-url that referenced this pull request Jan 27, 2026
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 -->
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