Skip to content

Commit 1c553c3

Browse files
authored
Update manifest examples to show origin scheme (#26)
Currently the "https://" scheme is required by the `origin` fields in `url_handlers`. This is consistent with Chromium's manifest parser implementation. This change updates the manifest examples in this explainer to reflect that. If there is feedback, we could consider also allowing a short form of origin strings where the "https://" scheme is implied.
1 parent f8af6a7 commit 1c553c3

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

explainer.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -91,13 +91,13 @@ Example web app manifest at `https://contoso.com/manifest.json` :
9191
"capture_links": "existing_client_event",
9292
"url_handlers" : [
9393
{
94-
"origin": "contoso.com"
94+
"origin": "https://contoso.com"
9595
},
9696
{
97-
"origin": "conto.so"
97+
"origin": "https://conto.so"
9898
},
9999
{
100-
"origin": "*.contoso.com"
100+
"origin": "https://*.contoso.com"
101101
}
102102
]
103103
}
@@ -119,13 +119,13 @@ Example web app manifest at `https://partnerapp.com/manifest.json`
119119
"capture_links": "existing_client_event",
120120
"url_handlers": [
121121
{
122-
"origin": "contoso.com"
122+
"origin": "https://contoso.com"
123123
},
124124
{
125-
"origin": "conto.so"
125+
"origin": "https://conto.so"
126126
},
127127
{
128-
"origin": "*.contoso.com"
128+
"origin": "https://*.contoso.com"
129129
}
130130
]
131131
}

0 commit comments

Comments
 (0)