Skip to content

/ servo

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

Jump to bottom

Implement Request::origin correctly #14504

Closed
Ms2ger opened this issue Dec 8, 2016 · 2 comments
Closed

Implement Request::origin correctly #14504

Ms2ger opened this issue Dec 8, 2016 · 2 comments
Labels
A-network

Comments

@Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented Dec 8, 2016

RequestInit stores a Url rather than an origin, which seems weird. We also don't handle Client at all (we should do that in the script thread anyway). Some code in https://github.com/Ms2ger/servo/tree/origin-request.

Also kinda depends on implementing Documents origins correctly.
@Ms2ger Ms2ger added the A-network label Dec 8, 2016
@Ms2ger Ms2ger mentioned this issue Dec 8, 2016
Closed
@fnune
Copy link
Contributor

@fnune fnune commented Jul 12, 2017

#16508
@nox
Copy link
Member

@nox nox commented Oct 7, 2017 

    pub origin: ImmutableOrigin,
@nox nox closed this Oct 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-network
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@Ms2ger @nox @fnune
You can’t perform that action at this time.