Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-43256

SSE event subscription should be organization aware

XMLWordPrintable

    • Blue Ocean - 1.1-beta-1, Blue Ocean - 1.1-beta2, Blue Ocean 1.1-beta4

      BlueOcean UI subscribes to SSE event without providing organization in the subscription filter.

      There are couple of places it needs to add jenkins_org in the sse subscription filter:

      For example: 

      this.jobListener = this.connection.subscribe('job', (event) => {
                this._handleJobEvent(event);
            }, {
    jenkins_org: 'org-1'
});

      See tfennelly comments for detail.

        1. Screenshot 2017-03-31 07.18.12.png
          Screenshot 2017-03-31 07.18.12.png
          164 kB

          [JENKINS-43256] SSE event subscription should be organization aware

          James Dumay created issue -
          James Dumay made changes -
          Epic Link New: JENKINS-35737 [ 171801 ]
          Tom FENNELLY made changes -
          Attachment New: Screenshot 2017-03-31 07.18.12.png [ 36838 ]
          Vivek Pandey made changes -
          Summary Original: SSE should be organization aware New: SSE event subscription should be organization aware
          Vivek Pandey made changes -
          Description Original: TBD Vivek to fill in New: BlueOcean UI subscribes to SSE event without providing organization in the subscription filter.

          There are couple of places it needs to add *jenkins_org* in the sse subscription filter:

          * [SseBus.js|https://github.com/jenkinsci/blueocean-plugin/blob/master/blueocean-core-js/src/js/sse/SseBus.js#L45]
          * [SSEService.js|https://github.com/jenkinsci/blueocean-plugin/blob/master/blueocean-core-js/src/js/services/SSEService.js#L11]

          For example:
          {code}
          this.jobListener = this.connection.subscribe('job', (event) => {
                          this._handleJobEvent(event);
                      }, {
              jenkins_org: 'org-1'
          });
          {code}

          See [~tfennelly] comments for detail.
          Michael Neale made changes -
          Priority Original: Minor [ 4 ] New: Major [ 3 ]
          Michael Neale made changes -
          Labels Original: cloudbees-internal-steel techical-debt New: cloudbees-internal-steel technical-debt
          Michael Neale made changes -
          Sprint Original: Blue Ocean 1.1 [ 271 ] New: Blue Ocean - techdebt sprint 1 [ 291 ]
          Vivek Pandey made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Vivek Pandey made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Michael Neale made changes -
          Sprint Original: Blue Ocean - 1.1-beta-1 [ 291 ] New: Blue Ocean - 1.1-beta-1, Blue Ocean - 1.1-beta2 [ 291, 306 ]

            vivek Vivek Pandey
            jamesdumay James Dumay
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: