Uploaded image for project: 'ZK'
  1. ZK
  2. ZK-4514

AuUploader and AuDropUploader exceptions are suppressed, need logging

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Normal Normal
    • 9.5.0
    • 9.0.1
    • None
    • Security Level: Jimmy
    • None
    • ZK 9.5.0 S1
    • None

      Debug Information

      https://github.com/zkoss/zk/blob/ec559349801fcc31e7cd5b2ccea14714fc249479/zk/src/org/zkoss/zk/au/http/AuUploader.java#L442

      AuUploader and AuDropUloader use common fileupload to retrieve uploaded medias during parseRequest()

       

      If the common-upload library encounter an issue during sfu.parseRequest(request), it may throw a FileUploadException

      This exception is caught in the try catch block located in the service method

      https://github.com/zkoss/zk/blob/ec559349801fcc31e7cd5b2ccea14714fc249479/zk/src/org/zkoss/zk/au/http/AuUploader.java#L167-L193

       

      In the exception is caught in this block, there is no client alert, or logging available for FileUploadException (it doesn't trigger any of the existing exceptions)

       

      Improvement

      Add a special handling case for FileUploadException which logs to logger

       

            rudyhuang rudyhuang
            MDuchemin MDuchemin
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - 1 day
                1d
                Remaining:
                Time Spent - 4 hours Remaining Estimate - 4 hours
                4h
                Logged:
                Time Spent - 4 hours Remaining Estimate - 4 hours
                4h