-
Notifications
You must be signed in to change notification settings - Fork 19
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
Sign access decision? #55
Comments
In which cases should the PEP not trust its designated PDP? Some level of "trust" could be provided by the transport layer, e.g. via TLS. Signing + validation implies notable efforts for both parties, whilst the whole process should be as fast as possible. As long as "Authorization Federation" isn't a clear concept (like with SAML-Federation) I would offload this to the transport layer and clearly assume mutual trust by the parties setting up both components. |
For Mutual TLS, I agree that it is a transport layer responsibility but we probably need to discuss in terms of patterns we want to support and if the API we provide can accommodate future use cases. Digitally signing can increase the trust in the authorization decision furthermore if mutual trust doesn't exist before the API call is made. I wouldn't see it as a mandatory API feature but something that is optional. Are all authorization decisions required to be fast? It is ideal but it depends on use cases we want to support. Also there is a spectrum of what is fast and near real time and what can take seconds, hours or could be tied to a workflow with manual interactions so I hope we can clarify that in the use cases. |
We can add a note in the draft "Security Considerations" section that reliable mechanisms must be used to ensure PDP - PEP communication integrity, privacy and authenticity (e.g. MTLS) |
@davidjbrossard did we include this in our Security Considerations? |
Should the access decision be signed by the PDP private key to ensure that the payload has not been tampered with?
The text was updated successfully, but these errors were encountered: