Uploaded image for project: 'CXF'
  1. CXF
  2. CXF-7462

OutboundSseEventImpl could use some minor tweaks

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.2.0
    • 3.2.1
    • JAX-RS
    • None
    • Unknown

    Description

      The OutboundSseEventImpl class could use some minor tweaks, including:
      1) Make the fields final to reflect that the event is immutable.
      2) Use defaults for the data type (String.class) and media type (SERVER_SENT_EVENT_TYPE).
      3) Restrict the constructor's visibility.

      I also plan to add some tests for these changes.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            andymc Andrew J McMurry
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment