filipw / Strathweb.AspNetCore.AzureBlobFileProvider

MIT License
54 stars 22 forks source link

Multitenancy #5

Open gsferreira opened 5 years ago

gsferreira commented 5 years ago

Hi @filipw !

In multi tenant scenarios, often we configure a container per tenant and have a single MVC application serving them using rooting.

What do you think about support a dynamic DocumentContainer?

filipw commented 5 years ago

yes that would make sense!

bencameron commented 5 years ago

Hi @filipw,

I have a similar need on one of my projects, in that I need to vary the container being used based on the request path. I've implemented it locally by introducing IContainerFactory, moving some logic from AzureBlobFileProvider into a DefaultContainerFactory, and then putting my own logic into a different implementation. Would you be interested in me submitting a pull request? I'm new to open source contributions, so I just want to make sure I'm following the right etiquette. :)

Thanks!

filipw commented 5 years ago

Would you be interested in me submitting a pull request? I'm new to open source contributions, so I just want to make sure I'm following the right etiquette. :)

If you can submit something, that would be great - thanks