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

SVG images not recogniced #37

Closed
Estartu opened this issue Nov 18, 2021 · 1 comment
Closed

SVG images not recogniced #37

Estartu opened this issue Nov 18, 2021 · 1 comment

Comments

@Estartu
Copy link

Estartu commented Nov 18, 2021

SVG images return mime_type='video/x-ms-asf'

cdgriffith added a commit that referenced this issue Feb 13, 2022
- Adding #37 SVG images (thanks to Gerhard Schmidt)
- Adding missing mimetypes for aac, vmdk, wmv and xcf
@cdgriffith cdgriffith mentioned this issue Feb 13, 2022
cdgriffith added a commit that referenced this issue Feb 13, 2022
- Adding #38 webp mimetype (thanks to phith0n)
- Adding #37 SVG images (thanks to Gerhard Schmidt)
- Adding missing mimetypes for aac, vmdk, wmv and xcf
@cdgriffith
Copy link
Owner

cdgriffith commented Feb 13, 2022

Thanks for the heads up! SVG was missing entirely, and probably because most regular magic number indexes don't include it as it's variable (it's just an XML file.)

I have include the lookup for three common places I have seen it. If you happen to come across one that doesn't match, please open the file in a text editor and just reply to this thrad with the part up to <svg .

For example one I found was:

<?xml version="1.0" standalone="yes"?>
<svg xmlns="http://www.w3.org/2000/svg" width="1" height="1">
<path style="fill:#ffffff; stroke:none;" d="M0 0L1 1L0 0z"/>
</svg>

Would just need:

<?xml version="1.0" standalone="yes"?>
<svg 

And of course now thinking about it probably just easier to add a footer lookup for </svg>, but maybe will add that in next release if this one starts becoming a headache :)

Added in 1.12

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

No branches or pull requests

2 participants