Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AMI remote file Mimetype (yes again) renames extensions #173

Closed
DiegoPino opened this issue Jul 5, 2023 · 2 comments
Closed

AMI remote file Mimetype (yes again) renames extensions #173

DiegoPino opened this issue Jul 5, 2023 · 2 comments
Assignees
Labels
CSV Processing Things we do here and there to keep the tabulated goddesses happy enhancement New feature or request File processing Everything is a file, even me. Release duty Things only a mad person should do
Milestone

Comments

@DiegoPino
Copy link
Member

What?

This is not an error per se. Using the normalized extension based on the actual remote Mime Type is OK. But for a specific need, 3D Models, where the exact name of a texture inside an Material file requires a 1:1 match this breaks things.

Need to debug exactly where this is happening bc reading my code I assumed I was already doing this, doing a reverse mapping from MimeType to extension, a normal from extension to Mime Type and only if both differed I used the calculated one. But there might be an IF or a logic conditional that is always ending in renaming. The specific case is JPG v/s JPEG

@DiegoPino DiegoPino added enhancement New feature or request Release duty Things only a mad person should do File processing Everything is a file, even me. CSV Processing Things we do here and there to keep the tabulated goddesses happy labels Jul 5, 2023
@DiegoPino DiegoPino added this to the 0.5.1 milestone Jul 5, 2023
@DiegoPino DiegoPino self-assigned this Jul 5, 2023
@DiegoPino
Copy link
Member Author

Gosh. This is auto-inflicted pain. See esmero/strawberryfield@8ebc80d
We are doing this to ourselves by trying to correct wrong remote Content-Types coming from Islandora ....

@DiegoPino
Copy link
Member Author

Resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CSV Processing Things we do here and there to keep the tabulated goddesses happy enhancement New feature or request File processing Everything is a file, even me. Release duty Things only a mad person should do
Projects
None yet
Development

No branches or pull requests

1 participant