Skip to content

Placeholder Submission with Proposed Fix for S3 Storage IDOR Vulnerability (Issue #137) #138

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

pravinkumar-exe
Copy link

This pull request includes a placeholder file with a proposed fix for the IDOR and Broken Access Control vulnerability reported in Issue #137.

…23#58)

This commit introduces a minimal placeholder file corresponding to Issue AIxBlock-2023#58, where an authenticated user can delete another user’s account using the /api/users/{user_id} endpoint.



Signed-off-by: pravinkumar-exe <[email protected]>
This commit adds a placeholder file as part of the bug submission process for issue AIxBlock-2023#116.

The file references the exposure of internal infrastructure details to normal users via the endpoint 'https://app.aixblock.io//api/settings/installation-service/'.

This is made from my forked branch to meet the required PR workflow. Looking forward to feedback.


Signed-off-by: pravinkumar-exe <[email protected]>
This is a placeholder submission with proposed fix for Issue AIxBlock-2023#137 (S3 IDOR vulnerability).

Signed-off-by: pravinkumar-exe <[email protected]>
…access on S3 config

This update adds a code-level fix for the IDOR vulnerability on the PUT /api/storages/s3-server/{id} endpoint.

It enforces role-based access control (RBAC), allowing only the owner of the storage configuration or an admin to perform updates.

All unauthorized users now receive a 403 Forbidden response. This resolves the access control issue described in Issue AIxBlock-2023#137.


Signed-off-by: pravinkumar-exe <[email protected]>
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

Successfully merging this pull request may close these issues.

1 participant