-
Notifications
You must be signed in to change notification settings - Fork 35
feat(context): introduce GroupRequest and ContextGroupId types for group management #2043
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
base: master
Are you sure you want to change the base?
Changes from 120 commits
c4a74ea
9fed2b3
073b252
5a3aa4e
787e4cd
35dc878
261b77c
53ac74d
cc28965
a11cc45
ee32493
a8269ef
2781dd3
d704238
f8b5908
5ed088c
cd1c7dc
9c7a209
83004c1
2a926f3
8bbb685
ddb8423
57c9221
b7d24e1
40a34ae
781c1ff
36404bd
2e6e71b
6dc04c5
badaf5c
e6253ba
c671613
c7b5f5a
384367b
71e5f43
a669c61
c52f3b3
92e3261
b70531d
0fa675b
e5643bf
f87bbea
13c1126
6fd4407
446740c
29bff4f
1889796
14b0708
55b505c
0a93f8d
f69394f
cbd2c7d
a4cee5b
af82536
6845238
d7c7d8f
89a5cec
a8ef7c7
375d35d
0d97589
f0b129b
bac3f1f
b429d75
3a03672
aad2273
cb19ab4
f821603
dcb4192
060b2bb
d2d66ba
3768dda
db06adc
159b70e
7aed757
1a33042
294b403
c7babef
839f8c2
d7555ff
b69fb22
7f32fdb
aa720f0
79d086f
37c6975
3fc2165
8f2a126
89fa042
f1396d9
1dcb00d
8fe6e15
0510256
e583faa
317cfbb
9263ff8
0814e2d
f4e7f7b
c5c129e
6ffcbcf
6d90322
db9ae5f
e7c8201
6e64680
b89cd59
3c9d30b
5e0e559
710d012
f99c63d
e509f0f
a3c42ea
df1c30e
d90d289
05033b9
9b40075
14f50e8
59980d4
de21a28
6dc4b02
76829bd
e5fc75b
de457cc
af83204
fc6adc2
525d22d
231bc27
125f298
7d3a2c3
b73d448
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -59,6 +59,14 @@ impl AuthService { | |
| self.token_manager.verify_token_from_headers(headers).await | ||
| } | ||
|
|
||
| /// Return the `public_key` field stored for `key_id`, if any. | ||
rtb-12 marked this conversation as resolved.
Show resolved
Hide resolved
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. 💡 Public key lookup by key_id should verify caller authorization The new Suggested fix: 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. 💡 Public key lookup should document authorization model The new Suggested fix: 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. 💡 Public key lookup by key_id could enable enumeration The new Suggested fix:
rtb-12 marked this conversation as resolved.
Show resolved
Hide resolved
rtb-12 marked this conversation as resolved.
Show resolved
Hide resolved
rtb-12 marked this conversation as resolved.
Show resolved
Hide resolved
rtb-12 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| /// | ||
| /// Used by the server auth guard to inject the authenticated identity into | ||
| /// request extensions so handlers can use it as the effective requester. | ||
| pub async fn get_key_public_key(&self, key_id: &str) -> Result<Option<String>, AuthError> { | ||
| self.token_manager.get_public_key_for_key_id(key_id).await | ||
rtb-12 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| } | ||
|
|
||
| /// Authenticate a token request | ||
| /// | ||
| /// This method authenticates the user using the provided token request | ||
|
|
||
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.
💡 New public key lookup should be audited for information disclosure
The new
get_key_public_keymethod exposes public keys by key_id; while public keys are generally safe to disclose, ensure this endpoint cannot be used to enumerate valid key_ids.Suggested fix: