nextcloud / tables

🍱 Nextcloud tables app
https://apps.nextcloud.com/apps/tables
GNU Affero General Public License v3.0
143 stars 24 forks source link

No error handling for MIME type check during import #1067

Closed ChristophWurst closed 4 months ago

ChristophWurst commented 4 months ago

Steps to reproduce

  1. Create a table
  2. Select a local text file for import

Expected behavior

Import dialog shows an error

Actual behavior

Import appears to take forever. Error is logged.

Bildschirmfoto vom 2024-05-06 11-16-39 Bildschirmfoto vom 2024-05-06 11-16-43

Tables app version

No response

Browser

No response

Client operating system

No response

Operating system

No response

Web server

None

PHP engine version

None

Database

None

Additional info

No response

juliushaertl commented 4 months ago

We should probably also accept text/plain if that is what a csv ends up with, though I thought we have a different mime type for that

luka-nextcloud commented 4 months ago

@ChristophWurst I cannot reproduce the problem, could you please share the file you tested?

ChristophWurst commented 4 months ago

Shared privately

luka-nextcloud commented 4 months ago

I can reproduce now, thanks.