Should a file bigger than the stated 8.5MB attempt to upload?

Bug #633760 reported by Charlie Amoah
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Invalid
Undecided
Ruslan Kabalin

Bug Description

Hello there, I tried uploading a file which was 40.5MB where it stated that the allowed file was 8.5MB. I was expecting an instant validation of "The file you are attempting to upload is bigger than allowed size" or something along those lines but it kept spinning until I stopped it.

Can someone cast an iron eye on that for me please? Here is the URL:http://master.dev.mahara.org/artefact/file/

Master Dev Mahara 1.3

Linux

FF3

Changed in mahara:
status: New → Confirmed
assignee: nobody → Ruslan Kabalin (ruslan-kabalin)
Revision history for this message
Ruslan Kabalin (rkabalin) wrote :

This bug is related to this one:
    https://bugs.launchpad.net/mahara/+bug/547505

I have applied several patches that assure proper file upload error handling and introduced MAX_FILE_SIZE hidden field wherever filebrowser pieform is used (this is applied to master, sorry for not mentioning this patch in the commit message):

    976451f83 filebrowser: File upload errors are not catched (bug #634147)
    2b6daab1c filebrowser: introduced MAX_FILE_SIZE hidden field
    88881404a Consider that post_max_size is the whole POST content (bug #634829)
    79e4e6bcb comment: add_feedback_form_submit function refactoring
    3dd0e821d pieforms: new filesize parameter in file-related elements
    461c86dbd comment: define MAX_FILE_SIZE for attachments
    f11ccc937d comment: exit from add_feedback_form_submit in case of quota exception

Though there is still a case when user will not get an error. If post-max-size error is triggered, it will not be processed properly. I have created another bug for this problem: https://bugs.launchpad.net/mahara/+bug/634193

Revision history for this message
Richard Mansfield (richard-mansfield) wrote :

I'm closing this bug now; it's addressed in the 3 new bugs Ruslan created (see last comment).

Changed in mahara:
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.