Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-9391

Set parameter enableEventsResending should be before Atmosphere wrap request

    Details

    • Estimated Complexity:
      Novice

      Description

      This is a defect regarding CAMEL-9364 feature.

      WebsocketConsumer has service method where Atmosphere handle incoming request. In case value for enableEventsResending will be set after request is handled OnOpen event will not be sent at all.

      Consequently, set this value should be before such as :
      this.enableEventsResending = enableEventsResending;
      framework.doCometSupport(AtmosphereRequest.wrap(request), AtmosphereResponse.wrap(response));

      In addition to this because of CamelWebSocketServlet use init method for set enableEventsResending value, tests which were written for this feature are wrong. To solve this a new TestSupport class should be created where init parameter will be set correctly and tests depending on this parameter will be moved to separate class which extend the new TestSupport class.

        Attachments

          Activity

            People

            • Assignee:
              davsclaus Claus Ibsen
              Reporter:
              pkletsko Pavlo Kletsko
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 1h
                1h
                Remaining:
                Remaining Estimate - 1h
                1h
                Logged:
                Time Spent - Not Specified
                Not Specified