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

Properties clean-up introduced content.upload.max regression

    XMLWordPrintable

Details

    • Bug
    • Status: Verified
    • Blocker
    • Resolution: Fixed
    • 10.0
    • 10.0
    • Config, Content

    Description

      The cleanup in SAK-21124, r105036 eliminated the default value for content.upload.max, which has been at 20 MB for quite some time. This results in two odd behaviors. The first is a limit of 1 MB and the second is that this number is omitted from the display.

      The related code is a bit kludgy, where there are two separate coded defaults: 1 and 20. There are also two confusingly named properties, content.upload.max and content.upload.ceiling.

      My suggestion is to change the code default to 20 in ResourcesHelperAction.java:1725, remove the "Default: 1" line in default.sakai.properties, and uncomment the actual setting.

      Gliffy Diagrams

        Zeplin

          Attachments

            Issue Links

              Activity

                People

                  aaronz Aaron Zeckoski (Inactive)
                  noahbotimer Noah Botimer
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                    Created:
                    Updated:
                    Resolved:

                    Git Integration