Skip to content
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

The client cannot tells whether audience restriction has been applied #55

Open
randomstuff opened this issue Sep 16, 2024 · 2 comments
Open

Comments

@randomstuff
Copy link

The current draft contains the wording:

While this attack is not explicitly enabled by this specification, and is possible in a plain OAuth 2.0 deployment, it is made somewhat more likely by the use of dynamically-configured clients. As such, the use of audience-restricted access tokens and Resource Indicators [RFC8707] is RECOMMENDED when using the features in this specification.

However there is currently no way for the OAuth client to know (in the general case i.e. loose coupling between the different actors) that the audience restriction has been properly supported and applied by authorization server. For example, the authorization server might silently ignore the resource parameter of the authorization request.

Ideally, the authorization server should probably include the resource back in the authorization response similar to how this is done for the scope parameter (and in a sense for the authorization_details parameter). Such an option should probably have been included in RFC8707, however.

Alternatively (or in addition) some authorization server metadata could be used to indicate that it supports resource indication.

@aaronpk
Copy link
Member

aaronpk commented Sep 23, 2024

I agree that the token response should include resource like it does scope to inform the client. However I don't think that is a mechanism that this spec can suggest or define, it probably should have been part of RFC8707. Same goes for the authorization server metadata to indicate support, that should have been part of 8707, and isn't something this spec can define.

@selfissued
Copy link
Collaborator

I agree with Aaron's assessment that this is out of scope for this specification. I'm prone to close this issue on that basis.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants