umbraco / Umbraco.StorageProviders

MIT License
30 stars 22 forks source link

Allow opting out of using Azure Blob for the image cache #25

Closed ronaldbarendse closed 2 years ago

ronaldbarendse commented 2 years ago

This PR adds a couple of overloads to make it easier to keep using the physical file ImageSharp.Web cache, fixing https://github.com/umbraco/Umbraco.StorageProviders/issues/15.

In a next major version, we might want to separate the configuration of the IFileSystem and ImageSharp.Web. Otherwise it would make sense to add default parameter values (useAzureBlobImageCache: true), so the new overloads can be removed again (adding optional parameters is a binary breaking change though).