wagtail-storages icon indicating copy to clipboard operation
wagtail-storages copied to clipboard

Potentially confusing name?

Open thibaudcolas opened this issue 5 years ago • 2 comments

Raised during team meeting – as I understand this is named as an analogy to django-storages, which supports multiple storage backends (popular cloud providers), while this project currently only supports S3.

A way to address this would be to simply rename the project. But if the general goal is to support more storage backends if possible, the existing name could work, just adding documentation to the README to explain the current support?

Something along the lines of,

## Supported storage providers

We currently only support AWS S3, as it has granular access control capabilities with ACLs. For other cloud providers,

- Azure has <limitations>, but they might be overcome.
- <other provider> has <more limitations>, and would not be currently usable

If you’re interested in contributing support for any of those providers, please open an issue to discuss implementation plans

If the confusion is a big issue it might also be worth updating the package’s description metadata to be suffixed with "– currently S3 only", or name drop S3 elsewhere in it.

thibaudcolas avatar Jan 31 '20 15:01 thibaudcolas

Hi Thibaud, I see you mentioned that "Azure has limitations, but they might be overcome". From what I can see, Azure Blob Storage only supports ACLs at the container level, not for individual files. It seems like Azure Data Lake Storage Gen2 has ACLS for individual files and can be seen as a superset of ABS, but django-storages doesn't explicitly support ADLS2. What do you think is the best way to overcome the Azure limitations and add it as a supported storage backend for wagtail-storages?

olifante avatar Nov 24 '22 15:11 olifante

@olifante 👋 I think my colleague @tm-kn is the mastermind here. I’ve only worked with Azure’s blob storage once myself, and have never looked into ADLS2. The limitation I mentioned back then I’m pretty sure was indeed the lack of ACL per object.

thibaudcolas avatar Dec 09 '22 23:12 thibaudcolas