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

Update ACL structures to have flexible types #958

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

smallhive
Copy link
Contributor

The PR is not solving test issues completely. It gives the ability to use wildcards in Principal and Action fields

api/handler/acl.go Outdated Show resolved Hide resolved
api/handler/json_types.go Outdated Show resolved Hide resolved
@smallhive smallhive force-pushed the 863-putbucketpolicy-principal branch from 09b4090 to cab784a Compare June 5, 2024 05:36
Copy link
Contributor Author

@smallhive smallhive left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extended docs

api/handler/acl.go Outdated Show resolved Hide resolved
api/handler/json_types.go Outdated Show resolved Hide resolved
@smallhive smallhive force-pushed the 863-putbucketpolicy-principal branch from cab784a to 187c658 Compare June 5, 2024 08:48
According to AWS docs https://docs.aws.amazon.com/IAM/latest/UserGuide/reference_policies_elements_principal.html
Action, Principal, Resource may vary by type. It can be string or slice of strings.

Signed-off-by: Evgenii Baidakov <[email protected]>
@smallhive smallhive force-pushed the 863-putbucketpolicy-principal branch from 187c658 to f4d62d6 Compare June 5, 2024 09:45
@roman-khimov roman-khimov merged commit 8fc4541 into master Jun 5, 2024
16 of 17 checks passed
@roman-khimov roman-khimov deleted the 863-putbucketpolicy-principal branch June 5, 2024 18:21
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.

PutBucketPolicy: specifying '*' as a principal results in 500
2 participants