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

Remove backwards compatibility support for deprecated inline IAM policies in S3 shares #1673

Open
dlpzx opened this issue Oct 30, 2024 · 0 comments

Comments

@dlpzx
Copy link
Contributor

dlpzx commented Oct 30, 2024

Is your idea related to a problem? Please describe.
In 2.3 IAM permissions needed for accessing S3 resources granted in s3-bucket-sharing and s3-access-points sharing were modified. Instead of using inline IAM policies, it introduced managed IAM policies (see PR: #1068). Currently if a share is using the old inline policy, data.all updates it to use managed policies if any sharing action happens on the share request (e.g. new items added to the share request).

There have been a number of releases between 2.3 and the next one 2.7 for users to get familiar with the new way of handling requestor permissions in data.all and to upgrade their current shares.

Describe the solution you'd like
I would like data.all codebase to be manageable and without dead code. For this reason I submit this request to remove the backwards compatibility code that supports inline IAM to Managed IAM policy migration.

P.S. Don't attach files. Please, prefer add code snippets directly in the message body.

@dlpzx dlpzx added this to v2.7.0 Oct 30, 2024
@github-project-automation github-project-automation bot moved this to Nominated in v2.7.0 Oct 30, 2024
@dlpzx dlpzx moved this from Nominated to This Sprint in v2.7.0 Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: This Sprint
Development

No branches or pull requests

1 participant