Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-32398

Remove uses of byte arrays when handling uploads

    XMLWordPrintable

    Details

    • Type: Feature Request
    • Status: Verified
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 11.4
    • Fix Version/s: 12.0
    • Component/s: Global
    • Labels:
      None
    • 11 status:
      Please Merge
    • Test Plan:
      Hide
      • Check that in a Dropbox you can upload files to multiple dropbox at once.
      • In Site Archive in Admin Workspace check that you can still batch import multiple archives at once (just need to actually check csv parsing, not the import).
      • Check import of CSV in calendar tool.
      • Create an announcement and add some attachments that are uploaded from the local machine.
      • Check that editing (create, edit, save, edit, save, check) of a HTML file works in resource.
      • Check that editing of a text file works in resources
      Show
      Check that in a Dropbox you can upload files to multiple dropbox at once. In Site Archive in Admin Workspace check that you can still batch import multiple archives at once (just need to actually check csv parsing, not the import). Check import of CSV in calendar tool. Create an announcement and add some attachments that are uploaded from the local machine. Check that editing (create, edit, save, edit, save, check) of a HTML file works in resource. Check that editing of a text file works in resources

      Description

      There has been work to prevent Sakai from reading uploads into memory before but these have just addresses some areas and there remains some usages of putting the whole uploading on the heap.

      We should tidy up the remaining places where the upload is read into memory so that the risk from allowing large uploads is smaller.

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                buckett Matthew Buckett
                Reporter:
                buckett Matthew Buckett
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Git Source Code