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

Redirect and credential URLs #660

Closed
youennf opened this issue Jan 10, 2018 · 5 comments
Closed

Redirect and credential URLs #660

youennf opened this issue Jan 10, 2018 · 5 comments
Labels
interop Implementations are not interoperable with each other

Comments

@youennf
Copy link
Collaborator

youennf commented Jan 10, 2018

I investigated how much redirect URLS with credentials are interoperable, and here is my current understanding of the landscape here based on web-platform-tests/wpt#8976:

  • Firefox is following the spec (error if cors mode and load is cross origin)
  • Safari is stripping any credentials from a redirect URL
  • Chrome is erroring redirect URLs containing credentials following https://www.chromestatus.com/feature/5669008342777856.
    (not tested on Edge)
    3 browsers, 3 different behaviors, can we try to converge here?
    I was hoping WebKit could be aligned with the spec.
    It would be good to know the rationale behind Chrome behavior.
    Maybe the spec should be updated accordingly.
@wanderview
Copy link
Member

Just to clarify, is this talking about URL foo@bar style credentials or Request.credentials property?

@youennf
Copy link
Collaborator Author

youennf commented Jan 10, 2018

This is about URL foo@bar credentials

@annevk
Copy link
Member

annevk commented Jan 10, 2018

Note that this is under discussion in #465 (I even pinged you!). Somewhat surprising the chromestatus.com entry doesn't mention that.

@annevk
Copy link
Member

annevk commented Jan 10, 2018

cc @mikewest

@annevk annevk added the interop Implementations are not interoperable with each other label Jan 10, 2018
@youennf
Copy link
Collaborator Author

youennf commented Jan 11, 2018

Oh, did not remember that issue. Let's continue it in #465

@youennf youennf closed this as completed Jan 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
interop Implementations are not interoperable with each other
Development

No branches or pull requests

3 participants