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

EventPublisherPlugin adds execution listener twice on activity with boundary event

    • Icon: Bug Report Bug Report
    • Resolution: Duplicate
    • Icon: L3 - Default L3 - Default
    • None
    • 7.13.0, 7.14.0, spring-boot 3.4.4
    • spring-boot
    • None

      org.camunda.bpm.spring.boot.starter.event.PublishDelegateParseListener doesn't check if execution listener is already added to the given activity.

      It leads that execution listener is added several times for one activity with boundary events.

      Example of affected process: 

      Here in the debugger you case see all listeneres added to "Test Task":

       

        This is the controller panel for Smart Panels app

            [CAM-12868] EventPublisherPlugin adds execution listener twice on activity with boundary event

            Hello dtitov,

            Thank you for reaching out to us with this.
            Could you please share your process model xml so we could understand better the scenario. And could you please specify the Spring boot start version and the Camunda version that you are using.
            Would it be possible to share a minimal project that reproduces the issue?

            Best,
            Yana

            Yana Vasileva added a comment - Hello dtitov , Thank you for reaching out to us with this. Could you please share your process model xml so we could understand better the scenario. And could you please specify the Spring boot start version and the Camunda version that you are using. Would it be possible to share a minimal project that reproduces the issue? Best, Yana

            Dmitrii Titov added a comment - https://github.com/miiit/camunda-spring-boot-example/blob/main/src/test/java/dt/test/TestApplicationTest.java

            Dmitrii Titov added a comment - - edited

            Hello @yana.vasileva ,  

            The test project is on github. The link is attached.

            The versions are:

            Spring-Boot: (v2.2.4.RELEASE)
            Camunda BPM: (v7.12.0)
            Camunda BPM Spring Boot Starter: (v3.4.1)

             

            Best regards,

            Dmitrii

            Dmitrii Titov added a comment - - edited Hello @yana.vasileva ,   The test project is on github. The link is attached. The versions are: Spring-Boot: (v2.2.4.RELEASE) Camunda BPM: (v7.12.0) Camunda BPM Spring Boot Starter: (v3.4.1)   Best regards, Dmitrii

            Yana Vasileva added a comment - - edited

            Hello Dmitrii,

            Thank you for sharing the example, it was convenient to reproduce and debug the issue, highly appreciated.

            I can confirm the problem:
            During the parsing of the process the execution listeners are added once during parsing of the boundary event and a second time during the parse of the event definition. Adding listeners for the event definitions should not be performed.

            Are you interested in providing a pull request to fix the behavior? We will provide you help and guidance if necessary.

            Best regards,
            Yana

            Yana Vasileva added a comment - - edited Hello Dmitrii, Thank you for sharing the example, it was convenient to reproduce and debug the issue, highly appreciated. I can confirm the problem: During the parsing of the process the execution listeners are added once during parsing of the boundary event and a second time during the parse of the event definition. Adding listeners for the event definitions should not be performed. Are you interested in providing a pull request to fix the behavior? We will provide you help and guidance if necessary. Best regards, Yana

            Hello Yana,

            Yes, I'm intrested.
            Should I create pull request for master branch only for camunda-bpm-spring-boot-starter?

            Best regards,
            Dmitrii

            Dmitrii Titov added a comment - Hello Yana, Yes, I'm intrested. Should I create pull request for master branch only for camunda-bpm-spring-boot-starter? Best regards, Dmitrii

            Hi dtitov,

            That's great.
            Yes, please create the pull request only for master branch here: https://github.com/camunda/camunda-bpm-platform/tree/master/spring-boot-starter
            (We merged the spring boot repository to the platform.)
            Let me know in case of any questions.

            Best regards,
            Yana

            Yana Vasileva added a comment - Hi dtitov , That's great. Yes, please create the pull request only for master branch here: https://github.com/camunda/camunda-bpm-platform/tree/master/spring-boot-starter (We merged the spring boot repository to the platform.) Let me know in case of any questions. Best regards, Yana

            Dmitrii Titov added a comment - - edited

            Hello Yana,

            Here is the PR: https://github.com/camunda/camunda-bpm-platform/pull/1224

            Best regards,
            Dmitrii

            P.S. Happy New Year!

            Dmitrii Titov added a comment - - edited Hello Yana, Here is the PR: https://github.com/camunda/camunda-bpm-platform/pull/1224 Best regards, Dmitrii P.S. Happy New Year!

            I am closing this ticket in benefit of CAM-12844.

            dtitov Thank you once again for the contribution. The process of the PR could be tracked in github where I provided a couple of review hints and the release related information will keep track in CAM-12844.

            Yana Vasileva added a comment - I am closing this ticket in benefit of CAM-12844 . dtitov Thank you once again for the contribution. The process of the PR could be tracked in github where I provided a couple of review hints and the release related information will keep track in CAM-12844 .

              yana.vasileva Yana Vasileva
              dtitov Dmitrii Titov
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: