Skip to content
This repository has been archived by the owner on Nov 24, 2021. It is now read-only.

setting contentType properly #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SalehAlbuga
Copy link

contentType was not being set properly, it was falling back to application/octet-stream when checking the files in Azure Storage Explorer, which may cause some issues with some file types like images when you load them from the URL. contentType is part of contentSettings object.
https://docs.microsoft.com/en-us/javascript/api/azure-storage/azurestorage.services.blob.blobservice.blobservice.createblobrequestoptions?view=azure-node-latest#contentsettings

contentType was not being set properly, it was falling back to application/octet-stream, which may cause some issues with some file types when you download the image. contentType is part of contentSettings object.
https://docs.microsoft.com/en-us/javascript/api/azure-storage/azurestorage.services.blob.blobservice.blobservice.createblobrequestoptions?view=azure-node-latest#contentsettings
@andrewhosgood
Copy link

Just realised this is a duplicate of my PR! #4

Same solution, different documentation 😂

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants