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

Mixed content level 2 #420

Closed
3 of 5 tasks
estark37 opened this issue Sep 6, 2019 · 6 comments
Closed
3 of 5 tasks

Mixed content level 2 #420

estark37 opened this issue Sep 6, 2019 · 6 comments
Assignees
Labels
Mode: none Does not require TAG review

Comments

@estark37
Copy link

estark37 commented Sep 6, 2019

こんにちはTAG!

I'm requesting a TAG review of:

Further details:

You should also know that...

This change has been running in Chrome as an experiment for several months (currently at 50% of beta channel).

We'd prefer the TAG provide feedback as (please select one):

  • open issues in our GitHub repo for each point of feedback
  • open a single issue in our GitHub repo for the entire review
  • leave review feedback as a comment in this issue and @-notify [github usernames]
@dbaron
Copy link
Member

dbaron commented Sep 6, 2019

I think an explainer would be useful, to answer the questions of what's new in level 2, and why are you doing a level 2 at all? Presumably there are underlying user needs that motivate that?

Also, I'd note that you ask for feedback in github issues, but the spec draft itself asks for feedback as email. Which is preferred?

@dbaron
Copy link
Member

dbaron commented Sep 6, 2019

And, for what it's worth, I'm fine with the explainer being in the spec introduction, but right now I think there's only a single sentence of explainer there, at least in terms of "what's new in level 2" rather than "what's in either level 1 or level 2":

This specification updates and extends [MIXED-CONTENT] to provide better security and privacy guarantees while minimizing breakage.

It seems like the rest of the introduction is describing level 1 or 2... although I might be misreading it.

@estark37
Copy link
Author

estark37 commented Sep 6, 2019

Thanks for the quick feedback! I expanded the introduction to more clearly explain what user problem we're trying to solve (lack of confidentiality and integrity for subresources, poor security UX), what's new in level 2, and to mention some alternatives we considered. Please let me know what you think.

@estark37
Copy link
Author

estark37 commented Sep 6, 2019

Also, I'd note that you ask for feedback in github issues, but the spec draft itself asks for feedback as email. Which is preferred?

Either is fine, and I updated the spec to reflect that.

@hadleybeeman
Copy link
Member

Thanks, @estark37! Much more helpful intro now. I can see what you're trying to do and why!

Note on 5.1: You might want to amend the text to say "...directive is not obsolete because it also allows developers to upgrade blockable content". As it is, we thought it sounds like this note was ignoring optionally-blockable content.

We appreciate the risk you've outlined in section 6. It was on our minds too. It's the same risk as in upgrade-insecure-requests.

This is a logical step on the path to the HTTPS-only web. Thanks for sharing it with us!

@kleinkk76
Copy link

こんにちはTAG!

I'm requesting a TAG review of:

Further details:

You should also know that...

This change has been running in Chrome as an experiment for several months (currently at 50% of beta channel).

We'd prefer the TAG provide feedback as (please select one):

  • open issues in our GitHub repo for each point of feedback
  • open a single issue in our GitHub repo for the entire review
  • leave review feedback as a comment in this issue and @-notify [github usernames]

#924

@rhiaro rhiaro added Mode: none Does not require TAG review and removed Progress: unreviewed labels May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mode: none Does not require TAG review
Projects
None yet
Development

No branches or pull requests

7 participants