-
Notifications
You must be signed in to change notification settings - Fork 76
S3: add NetApp StorageGrid CORS configuration details #874
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
base: master
Are you sure you want to change the base?
S3: add NetApp StorageGrid CORS configuration details #874
Conversation
* Added instructions for configuring CORS in NetApp StorageGrid to enable file uploads and previews. * applying S3 configs seems to not be sufficent for StorageGrid
|
We had the same problem and provided the following PR: #867 However, based on our testing, such extensive permissions were not necessary. Can anyone verify this? |
|
Thanks for the cross-reference @m6121 To clarify: the config in your PR is correct for AWS S3, which is more permissive and doesn’t require declaring every method/header explicitly for multipart uploads. Having said that, StorageGrid behaves differently; it enforces stricter CORS validation and fails multipart/pre-signed upload flows unless all relevant methods and headers (e.g., So your configuration is valid for AWS, and the expanded example here is needed for StorageGrid and similar S3-compatible backends. We'll keep both documented so users on different platforms have clear guidance. Happy to hear others’ experiences as well. |
|
Hi @Samk13 Actually, we also use Netapp Storagegrid and failed in enabling multipart upload due to the missing permissions. However, instead of enabling all methods, we tried to find a working solution which was the result in our PR (just adapted to the existing AWS configuraiton style). This setup seems to work in our tests with our netapp storage. However, we are not exactly sure, if the other methods in this PR |
|
Interesting, thanks for the details! I'll check with our DevOps team on whether those extra methods were required in our setup or if a narrower rule set worked for us too. |
|
That's great. We are also not sure whether there are other side-effects of these more restrictive rules. Looking forward to your feedback. |
Hello, I'm the DevOps team! ✋ 😅 I need to double check something in our proxy first, just so I don't confuse matters more than necessary. BRB. |
|
Confusingly, we have two different policies in place. I'll have to experiment next week to verify what's really required. |
❤️ Thank you for your contribution!
Description
to enable file uploads and previews.
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:
masterbranch.productionbranch following approval or indicate to a maintainer that it should be backported.Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that: