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.

Sign up for GitHub

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

Jump to bottom

Percent encode fragments too #169

Merged
merged 1 commit into from
Dec 9, 2016
Merged

Percent encode fragments too #169

merged 1 commit into from
Dec 9, 2016

Conversation

annevk
Copy link
Member

@annevk annevk commented Dec 8, 2016

Fixes #150.

@annevk
Copy link
Member Author

annevk commented Dec 8, 2016

web-platform-tests/wpt#4298 has test changes. Make sure to mention those when committing.

@annevk annevk merged commit 373dbed into master Dec 9, 2016
@frewsxcv frewsxcv mentioned this pull request Dec 9, 2016
Closed
Manishearth added a commit to Manishearth/rust-url that referenced this pull request Dec 10, 2016
@Manishearth
Percent-encode fragments based on whatwg/url#169
a3e8a6d 
fixes servo#246
@Manishearth Manishearth mentioned this pull request Dec 10, 2016
Merged
bors-servo pushed a commit to servo/rust-url that referenced this pull request Dec 15, 2016
Auto merge of #248 - Manishearth:percent-fragment, r=SimonSapin
37fe8a7 
Percent-encode fragments while parsing

Based on whatwg/url#169

Fixes #246

cc @valenting

r? @SimonSapin

<!-- 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/248)
<!-- Reviewable:end -->
@annevk annevk deleted the annevk/non-Unicode-fragments branch December 28, 2016 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@annevk