-
Notifications
You must be signed in to change notification settings - Fork 4
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
Limiting Proof of Possession Scope #50
Comments
This is related to the bigger question of whether the token is reusable or associated with a single HTTP request. |
The But also what @yaronf said |
and the |
From what I can see we have the following options for the Workload Proof Token (DPoP approach). The order is random and the options are not pre-filtered.
I'm sure there's more options that don't come into my mind at the moment. |
B and D are clearly a no go because they leave to much to the user and/or do not promote interoperability. Personally I think we'll be forced into C, I think the complexity is justified and people will be writing new code for WIMSE anyway. I don't understand the comment about OAuth in option E. To me this is not OAuth DPoP, this is an entirely new think inspired by that DPoP. And yes, features we add here may be "back ported" into OAuth DPoP. |
@yaronf yes, this is what I was trying to say, if WIMSE would define claims for messaging queue possession, OAuth could profit from that too. Anyway, nothing we should put a lot of weight on IMO. |
Closing this in favour of #39. Overall, the authors believe that including the user context to the WPT scope makes sense. The current draft includes audience, ath, tth in the DPoP approach and allows signature over url, header & body in the Http Message Signature approach. #39 is focusing on accustom deployments that may use over approaches with the WPT token. |
Commenting as identity enthusiast as opposed to WIMSE co-chair:
Do we need additional mechanism defined in the Workload Proof Token for additional scoping (e.g. specific API on a target workload)?
The text was updated successfully, but these errors were encountered: