-
Notifications
You must be signed in to change notification settings - Fork 162
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
Describe manifest update behavior #1011
Merged
Merged
Changes from 13 commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
9534039
Describe manifest update behavior with app identity
philloooo 1473290
fix missing section
philloooo 482259b
Move privacy recommendations to second paragraph as recommended in pr…
philloooo fe8f846
Merge branch 'gh-pages' into update
marcoscaceres 2b32672
Update index.html
philloooo a80d96f
Merge branch 'w3c:gh-pages' into update
philloooo ff204a0
address comments
philloooo b27c7ec
Merge branch 'gh-pages' into update
marcoscaceres a7231a4
Merge branch 'gh-pages' into update
marcoscaceres 4e64aa4
Update index.html
philloooo 4669fbf
Update index.html
philloooo c67e456
fix syntax
philloooo 7f7f832
add some context to partial update
philloooo 751e534
Update index.html
philloooo ca2e788
Remove report option
philloooo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
|
@@ -842,11 +842,12 @@ <h3> | |||||||||
application, it SHOULD treat that manifest as a description of a | ||||||||||
distinct application, even if it is served from the same URL as that | ||||||||||
of another application. When the user agent sees a manifest where | ||||||||||
|manifest|["id"] [=url/equal|equals=] the [=identity=] of an | ||||||||||
already-installed application, it SHOULD be used as a signal that | ||||||||||
this manifest is a replacement for the already-installed | ||||||||||
application's manifest, and not a distinct application, even if it is | ||||||||||
served from a different URL than the one seen previously. | ||||||||||
|manifest|["id"] is [=url/equal=] with [=URL serializer/exclude | ||||||||||
fragment|exclude fragment true=] to the [=identity=] of an | ||||||||||
already-installed application, it SHOULD be used as a signal that this | ||||||||||
manifest is a replacement for the already-installed application's | ||||||||||
manifest, and not a distinct application, even if it is served from a | ||||||||||
different URL than the one seen previously. | ||||||||||
</p> | ||||||||||
<p class="note"> | ||||||||||
The [=identity=] can be used by a service that collects lists of web | ||||||||||
|
@@ -1293,6 +1294,13 @@ <h3> | |||||||||
<li>[=Process the `id` member=] passing |json|, |manifest| and | ||||||||||
|document URL|. | ||||||||||
</li> | ||||||||||
<li> | ||||||||||
If the [=document=]'s [=document|processed | ||||||||||
manifest=] is not null, and [=document=]'s [=document|processed | ||||||||||
manifest=]'s id is not | ||||||||||
[=URL/equal=] with [=URL serializer/exclude fragment|exclude fragment true=] to |manifest|["id"], | ||||||||||
return. | ||||||||||
</li> | ||||||||||
<li>[=Process the `scope` member=] passing |json|, |manifest|, and | ||||||||||
|manifest URL|. | ||||||||||
</li> | ||||||||||
|
@@ -1421,7 +1429,57 @@ <h3 id="applying"> | |||||||||
<h3> | ||||||||||
Updating the manifest | ||||||||||
</h3> | ||||||||||
<aside class="issue" data-number="446"></aside> | ||||||||||
<p> | ||||||||||
As specified for [^link/rel/manifest^] link relation, the manifest | ||||||||||
is fetched and processed on every page load. When the [=processing | ||||||||||
a manifest=] is successful, user agents MAY apply updated manifest | ||||||||||
to any current and future <a>application contexts</a> associated | ||||||||||
with the application. | ||||||||||
</p> | ||||||||||
<p> | ||||||||||
For the purpose of updating, the following member are | ||||||||||
<dfn>security-sensitive members</dfn>, as they are presented during | ||||||||||
installation and on launch surfaces: | ||||||||||
</p> | ||||||||||
<ol> | ||||||||||
<li>[=manifest/short_name=], | ||||||||||
</li> | ||||||||||
<li>[=manifest/icons=] | ||||||||||
</li> | ||||||||||
<li>[=manifest/name=], | ||||||||||
</li> | ||||||||||
</ol> | ||||||||||
<p data-cite="permissions"> | ||||||||||
User agents SHOULD NOT automatically apply changes to | ||||||||||
[=security-sensitive members=] without [=express permission=] from | ||||||||||
the user. | ||||||||||
</p> | ||||||||||
<p> | ||||||||||
Instead, user agents SHOULD present changes to [=security-sensitive | ||||||||||
members=] with appropriate management options, so the user can make | ||||||||||
an informed decision about updating the web application. | ||||||||||
</p> | ||||||||||
<p> | ||||||||||
The user agent MAY automatically apply the changes if the update | ||||||||||
does not contain changes to [=security-sensitive members=]. | ||||||||||
</p> | ||||||||||
<aside class="note" title= | ||||||||||
"A user agent will not apply a partial update"> | ||||||||||
<p> | ||||||||||
When the update contains changes both to [=security-sensitive | ||||||||||
members=] and other members. A user agent SHOULD not automatically | ||||||||||
philloooo marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||
partially update. For example, the user agent could present | ||||||||||
options to the user: | ||||||||||
</p> | ||||||||||
philloooo marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||
<ol> | ||||||||||
<li>Accept the update | ||||||||||
</li> | ||||||||||
<li>Uninstall the web app, or | ||||||||||
</li> | ||||||||||
<li>Report the website as abusive/spam. | ||||||||||
</li> | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||
</ol> | ||||||||||
</aside> | ||||||||||
</section> | ||||||||||
</section> | ||||||||||
</section> | ||||||||||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marcoscaceres I've added some context to the partial update note, let me know what you think.