Uploaded image for project: 'Opencast'
  1. MH-12451

Lock file utility method should throw exceptions

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed and reviewed
    • Affects versions: 4.0
    • Fix versions: 4.0
    • Components: Backend Software
    • Labels:
      None
    • Severity:
      Incorrectly Functioning Without Workaround
    • Steps to reproduce:
      Hide
      The IoSupport locked() method should propagate exceptions and not wrap them in a RuntimeException. The caller should be able to catch and handle it properly.

      We've seen some errors in the log, where the workspace get() method throws a runtime exception by creating a lock for the file that should be downloaded. This exception type must not be declared in Java, so this exception is nowhere catched and handled properly.

      We should avoid to throw RuntimeExceptions where ever possible!
      Show
      The IoSupport locked() method should propagate exceptions and not wrap them in a RuntimeException. The caller should be able to catch and handle it properly. We've seen some errors in the log, where the workspace get() method throws a runtime exception by creating a lock for the file that should be downloaded. This exception type must not be declared in Java, so this exception is nowhere catched and handled properly. We should avoid to throw RuntimeExceptions where ever possible!

      TestRail: Results

        Attachments

          Activity

            People

            • Assignee:
              waldemarsmirnow Waldemar Smirnow
              Reporter:
              waldemarsmirnow Waldemar Smirnow
            • Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                TestRail: Cases