Skip to content

Commit f5490e8

Browse files
jcjonesjgraham
authored andcommitted
Add [SecureContext] to navigator.credentials
It was neglected to mark navigator.credentials as [SecureContext], yet it must be for spec compliance and powerful-features compliance. MozReview-Commit-ID: BYKGqqhoS2L Upstreamed from https://bugzilla.mozilla.org/show_bug.cgi?id=1430947 [ci skip]
1 parent 72f8f50 commit f5490e8

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
<!doctype html>
2+
<meta charset=utf-8>
3+
<title>Test that Credential Management requires secure contexts</title>
4+
<link rel="help" href="https://w3c.github.io/webappsec-credential-management/#idl-index">
5+
<script src=/resources/testharness.js></script>
6+
<script src=/resources/testharnessreport.js></script>
7+
<script>
8+
"use strict";
9+
test(() => {
10+
assert_false(isSecureContext);
11+
assert_false('credentials' in navigator);
12+
}, "Credential Management must not be accessible in insecure contexts");
13+
</script>

0 commit comments

Comments
 (0)