search

gaul / s3proxy

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

Make ServletRequest available to BlobStoreLocator #260

Open szquadri opened 6 years ago

szquadri commented 6 years ago

To help make s3proxy more generic and suitable for custom or embedded usage, it will be better if locateBlobStore method of interface BlobStoreLocator is passed current ServletRequest so it can take more complex decision in locating suitable BlobStore based on factors like hostname / virtualhost or any other factor/info contained in ServletRequest.

This will help implement, for example, a multi-tenant blob store/source.

gaul commented 6 years ago

This sounds reasonable. Can you submit a pull request for this?

szquadri commented 6 years ago

Sure. I will be submitting a pull request.