-
Notifications
You must be signed in to change notification settings - Fork 84
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
Adds proxy server to minio / aws s3 storage #962
Adds proxy server to minio / aws s3 storage #962
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MaxFBurg 🚀 Awesome job in finding a solution for proxy mode with S3! 🚀
There are a few minor items that were not considered here but they are mainly related to tests and code coverage. Offered suggestions to make it easy to get this in but other than that this looks great!
… test Co-authored-by: Raphael Guzman <[email protected]>
Co-authored-by: Raphael Guzman <[email protected]>
@guzman-raphael Thank you very much for your comments and the very nice feedback and helpful explanations, I highly appreciate that! I committed all your suggestions - just let me know if you need anything else. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MaxFBurg Not a problem! 🤝
Almost there; just need to fix a small typo that I made in my previous suggestion. 😅
Co-authored-by: Raphael Guzman <[email protected]>
Co-authored-by: Raphael Guzman <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MaxFBurg Thanks for fixing these updates and ...
🎉 Congrats on your first contribution! 🎉
Thank you very much @guzman-raphael ! |
This PR adds the possibility to configure a proxy server for the mini / aws s3 storage. The PR resolves #961 .