gaul / s3proxy

Access other storage backends via the S3 API
Apache License 2.0
1.65k stars 220 forks source link

treat null and empty string equally for endpoint properties #570

Closed larshagencognite closed 8 months ago

larshagencognite commented 8 months ago

This improves the usability of the docker image, as it defaults to empty strings for both properties. With this fix, the docker image will work when only one of the endpoints are set.

larshagencognite commented 8 months ago

Alternative to https://github.com/gaul/s3proxy/pull/557

larshagencognite commented 8 months ago

Fixes https://github.com/gaul/s3proxy/issues/523, https://github.com/gaul/s3proxy/issues/528 and https://github.com/gaul/s3proxy/issues/556

gaul commented 8 months ago

Thank you for your contribution @larshagencognite!