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

xls files need 521 bytes #174

Open
vgambier-altus opened this issue Apr 10, 2024 · 1 comment
Open

xls files need 521 bytes #174

vgambier-altus opened this issue Apr 10, 2024 · 1 comment

Comments

@vgambier-altus
Copy link

The documentation says to pass the first 261/262 bytes. However, after some testing, it seems that for .xls files, 261 bytes is not enough. Passing 520 bytes is not enough either. Passing 521 bytes is enough, and .xls files are correctly identified as .xls files.

This issue does not affect .xlsx files, which are correctly identified with only 261 bytes.

I'm not sure if this is a documentation issue or an implementation issue.

@CatKasha
Copy link
Contributor

readme is outdated, filetype.py reads 8192 bytes and should detect xls and xlsx files
b71bf17

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