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

Fileupload .csv transforms to .txt #9314

Closed
leganz opened this issue Jan 12, 2024 · 4 comments · Fixed by #9389
Closed

Fileupload .csv transforms to .txt #9314

leganz opened this issue Jan 12, 2024 · 4 comments · Fixed by #9389
Labels

Comments

@leganz
Copy link
Contributor

leganz commented Jan 12, 2024

Bug description

Sind #9033 and Statamic 3.4.15 uploading a csv file transforms the file to a .txt

How to reproduce

Upload the following file to
access-code.csv the assets.

or look at this video: https://share.cleanshot.com/nKVFtMmq5gwyr9f41L8Q

Logs

No response

Environment

Environment
Application Name: Statamic
Laravel Version: 9.52.16
PHP Version: 8.2.10
Composer Version: 2.6.3
Environment: local
Debug Mode: ENABLED
URL: localhost:8006
Maintenance Mode: OFF

Installation

Other (please explain)

Antlers Parser

None

Additional details

Running Statamic 3.4.15

@duncanmcclean
Copy link
Member

This is intentional for security reasons.

You can workaround it though. See this comment from a previous issue: #9182 (comment)

@jasonvarga
Copy link
Member

jasonvarga commented Jan 12, 2024

This only seems to happen with some CSVs. Probably if the contents do not immediately look like a CSV, like if there's not a header row at the start.

Can you show the first line of the file?

@leganz
Copy link
Contributor Author

leganz commented Jan 23, 2024

Sorry for the late response
The csv contents first 3 lines

Datum;Serie;Staffel;Genre;Sender
02.01.2024;White Collar;1;Krimi;Warner TV Serie Germany
02.01.2024;Picture Perfect Mysteries;1;Krimi;Super RTL

@jasonvarga
Copy link
Member

Ah it's not picking it up as a CSV (Comma Separated Values) because it's not comma separated.

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

Successfully merging a pull request may close this issue.

3 participants