Skip to content
This repository was archived by the owner on May 23, 2022. It is now read-only.

Commit 3d0c836

Browse files
Apply suggestions from code review
Co-authored-by: Brian Lin <[email protected]>
1 parent 4479a26 commit 3d0c836

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

docs/security/tokens/overview.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
DateReviewed: 2022-03-10
2-
=======
32
Bearer Token Overview
43
=====================
54

@@ -36,7 +35,7 @@ SciTokens and WLCG Tokens are similar standards and have some common claims:
3635

3736
The issuer identifies the organization that issued the token.
3837
An issuer looks like an HTTPS URL;
39-
this URL must be valid and publicly accessible because it is used by services to validate the token.
38+
this URL must be valid and publicly accessible as they are used by site services to validate the token.
4039
Token issuers will be described below.
4140

4241
**Subject ("sub")**
@@ -180,7 +179,7 @@ If an incoming (pre-routed) pilot on a CE has a token, it will have the followin
180179
| AuthTokenIssuer | The URL of the issuer of the token |
181180
| AuthTokenScopes | Any scope restrictions on the token |
182181
| AuthTokenSubject | The `sub` claim of the token |
183-
| AuthTokenGroups | The `wlcg.groups` claim of the token |
182+
| AuthTokenGroups | The `wlcg.groups`, if any, claim of the token |
184183

185184
(A pre-routed job is a job without `RoutedJob=True` in its classad.)
186185

0 commit comments

Comments
 (0)