-
Notifications
You must be signed in to change notification settings - Fork 73
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
feat(jans-cedarling): Implement python bindings for the authorize method #9731
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Oleh Bohzok <[email protected]>
Signed-off-by: Oleh Bohzok <[email protected]>
Signed-off-by: Oleh Bohzok <[email protected]>
Signed-off-by: Oleh Bohzok <[email protected]>
Signed-off-by: Oleh Bohzok <[email protected]>
…rceData method from_dict Signed-off-by: Oleh Bohzok <[email protected]>
Signed-off-by: Oleh Bohzok <[email protected]>
…optimization related to type conversion Signed-off-by: Oleh Bohzok <[email protected]>
Signed-off-by: Oleh Bohzok <[email protected]>
… authorize method Signed-off-by: Oleh Bohzok <[email protected]>
Signed-off-by: Oleh Bohzok <[email protected]>
Signed-off-by: Oleh Bohzok <[email protected]>
DryRun Security SummaryThe pull request covers a wide range of updates to the Cedarling project, with a focus on improving the authorization-related functionality, including policy store configuration, authorization request handling, error handling, and documentation and examples. Expand for full summarySummary: The code changes in this pull request cover a wide range of updates to the Cedarling project, with a focus on the authorization-related functionality. The key changes include:
From an application security perspective, these changes appear to be focused on improving the overall security and reliability of the Cedarling project. The introduction of more granular and context-aware policies, the handling of authorization requests, and the robust error handling mechanisms are all positive steps towards a secure authorization system. However, it's important to review the implementation details of these changes, especially in areas like input validation, data handling, and the use of external libraries, to ensure that there are no unintended security vulnerabilities introduced. Files Changed:
Code AnalysisWe ran Riskiness🟢 Risk threshold not exceeded. |
Waiting to close After that, I will add type hints for current changes |
Prepare
Description
Target issue
link
closes #9706
Implementation Details
added all entities for
authorize
method, including errorsTest and Document the changes
Please check the below before submitting your PR. The PR will not be merged if there are no commits that start with
docs:
to indicate documentation changes or if the below checklist is not selected.