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

Explain why only https urls should be associated. #19

Open
LuHuangMSFT opened this issue Oct 30, 2020 · 1 comment
Open

Explain why only https urls should be associated. #19

LuHuangMSFT opened this issue Oct 30, 2020 · 1 comment

Comments

@LuHuangMSFT
Copy link
Collaborator

This issue originated from a CL review discussion here: https://chromium-review.googlesource.com/c/chromium/src/+/2405696

The explainer can be more explicit about why only https origins are allowed to be required in url_handlers for URL handling.


Notes:

  • There is an expectation that content in PWAs come from https URLs.
  • We have not received any requests for http support.
  • We can provide better security guarantees if only https URLs are allowed here.
    • If the association file is fetched from a http URL, it cannot be trusted to represent the consent of content owners.
@SamB
Copy link

SamB commented Feb 14, 2024

What should happen if the user opens an old http URL that now redirects to https?

What if they opened it from outside the browser?

What if there's an MITM sending the wrong redirects?

It's certainly true that you don't want to fetch association files from plain http, but it should be fine to just use the https one for the same domain, shouldn't it?

It's also plausible that asking the PWA to handle an http URL would result in it accidentally making plain http requests.

Perhaps a mechanism to explicitly request that certain http:// URLs be rewritten to https:// before the PWA sees them?

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

No branches or pull requests

2 participants