View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
16941Bug reportsSurvey editingpublic2021-01-04 12:492021-02-25 14:16
ReporterJmantysalo Assigned To 
PrioritynormalSeverityminor 
Status confirmedResolutionopen 
Product Version4.4.0-RC1 
Summary16941: Too big file to upload, no error message
Description

When I try to upload for example .ps file I get message "Denied file extension." But if I try to upload too big file it just fails silently.

TagsNo tags attached.
Complete LimeSurvey version number (& build)Version 4.4.0-RC1+201223
I will donate to the project if issue is resolvedNo
Sync to Zoho ProjectYes
Browser
Database & DB-VersionN/A
Server OS (if known)
Webserver software & version (if known)
PHP VersionN/A

Activities

gabrieljenik

gabrieljenik

2021-02-01 15:17

developer   ~61846

I think mayb we need to port this?
16733
gabrieljenik

gabrieljenik

2021-02-18 21:46

developer   ~62368

Last edited: 2021-02-18 22:41

View 2 revisions

@c_schmitz, there is a situation when the file size is bigger than the allowed by the php.ini.
On those cases, the post is just empty. Then the error is that the CSRF balidation fails.

That's something we can;t handle.
My suggestion is to change the standard error message, mentioning that these errors could be due to big file sizes.

What do you think?
gabrieljenik

gabrieljenik

2021-02-18 21:46

developer   ~62369

Still, we will patch as there are some situations where the file is big, but not that big, where we can show a descriptive message.
gabrieljenik

gabrieljenik

2021-02-18 22:42

developer   ~62380

ping @c_schmitz
c_schmitz

c_schmitz

2021-02-19 00:48

administrator   ~62382

Sounds good. Make it so. :-)
gabrieljenik

gabrieljenik

2021-02-25 14:16

developer   ~62517

PR: https://github.com/LimeSurvey/LimeSurvey/pull/1783
Added it to the dev branch as to have more testing time

Issue History

Date Modified Username Field Change
2021-01-04 12:49 Jmantysalo New Issue
2021-01-28 20:06 cdorin Priority none => normal
2021-01-28 20:06 cdorin Status new => confirmed
2021-01-28 20:06 cdorin Sync to Zoho Project => Yes
2021-02-01 15:17 gabrieljenik Note Added: 61846
2021-02-18 21:46 gabrieljenik Note Added: 62368
2021-02-18 21:46 gabrieljenik Note Added: 62369
2021-02-18 22:41 gabrieljenik Note Edited: 62368 View Revisions
2021-02-18 22:42 gabrieljenik Note Added: 62380
2021-02-19 00:48 c_schmitz Note Added: 62382
2021-02-25 14:16 gabrieljenik Note Added: 62517