ActiveStorage: Non-image file is not deleted when its blob is destroyed about cloudinary_gem

[aleksandar-cloudinary]

Hi @zokioki - Thanks for reporting this - I'll go through it and share more details afterwards.

[const-cloudinary]

@zokioki , in addition to the PR mentioned above there is another PR (#508) that takes the content type parameter directly from the function, without passing it through the ActiveStorage::BlobKey, hopefully it should resolve the issue.
If not, please provide the csv file you are uploading and we can investigate it further. Mime type detection is not always accurate.
Thanks once again for raising the issue!

[const-cloudinary]

Hello @zokioki.

There is a fix for adding file extension for raw files:
#507
It will be merged soon.

It fixes the file extension problem.

Regarding resource_type: "image" for your CSV file.

Can you please provide your file?

We tried to reproduce it on our end and the resource_type was detected correctly.

Thanks!

[zokioki]

@const-cloudinary Thanks for the quick work on the fix.

It looks like my assumption about the resource_type mismatch was incorrect. When composing this issue, I assumed the key passed to the resource_type method was the blob's raw key string, but it appears that it is an instance of ActiveStorage::BlobKey, which contains the file's content_type and results in the appropriate resource type.

It seems that the extension logic was the only issue here.