Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-9096

Max tasks and lock duration are not validated correctly on fetch and lock for external tasks

    Details

    • Type: Bug Report
    • Status: Closed
    • Priority: L3 - Default
    • Resolution: Won't Fix
    • Affects Version/s: 7.10.0-alpha1, 7.10.0-alpha5
    • Fix Version/s: 7.8.x, 7.9.x, 7.10.x, 7.11.0, 7.11.0-alpha1
    • Component/s: engine
    • Labels:
      None

      Description

      Steps to reproduce

      1. Make sure you have some running external task instances
      2. Fetch and lock for external tasks either with
        • a negative lock duration
        • or max tasks less than zero

      Expected behavior
      An exception is thrown as these values are forbidden

      Observed behavior
      No exception is thrown

      Hint: ensure util constraints check wrong field names

        Issue Links

          Activity

          Show
          thorben.lindhauer Thorben Lindhauer added a comment - Tassilo Weidner , isn't this already covered by these test cases? https://github.com/camunda/camunda-bpm-platform/blob/7.10.0/engine/src/test/java/org/camunda/bpm/engine/test/api/externaltask/ExternalTaskServiceTest.java#L344-L358 https://github.com/camunda/camunda-bpm-platform/blob/7.10.0/engine/src/test/java/org/camunda/bpm/engine/test/api/externaltask/ExternalTaskServiceTest.java#L374-L388
          Hide
          tassilo.weidner Tassilo Weidner added a comment -

          I had a look into it and I can confirm that it behaves as expected. However, the field names in the exception messages are wrong. Therefore, I close this issue and created CAM-9576 to correct the error messages.

          Show
          tassilo.weidner Tassilo Weidner added a comment - I had a look into it and I can confirm that it behaves as expected. However, the field names in the exception messages are wrong. Therefore, I close this issue and created CAM-9576 to correct the error messages.

            People

            • Assignee:
              Unassigned
              Reporter:
              tassilo.weidner Tassilo Weidner
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development