[JENKINS-35153] Events.JobChannel#job_crud_deleted publishing blocked by AccessControl - Jenkins Jira

XML

Word

Printable

Details

Type: Bug
Status: Closed (View Workflow)
Priority: Minor
Resolution: Cannot Reproduce
Component/s: blueocean-plugin, sse-gateway-plugin
Labels:
- blueocean

Similar Issues:

Show
Epic Link:
SSE
Sprint:
1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, pannonian, iapetus

Description

This is actually an issue in the pubsub-light-module. The GuavaPubsubBus does an AccessControlled check before allowing publishing of any message. The first part of that is to check for the existence of the model object associated with the event. But of course In the case of this delete event (or any other delete event), the model object no longer exists, causing the security check to fail and for the event to not be published.

Need to come up with some event specific way of indicating that it's OK to forward the event if the model object does not exist.

Attachments

Activity

Ascending order - Click to sort in descending order

Tom FENNELLY created issue - 2016-05-26 14:23

James Dumay made changes - 2016-06-15 05:16

Field	Original Value	New Value
Labels		blueocean

James Dumay made changes - 2016-06-15 05:18

Epic Link

JENKINS-35737 [ 171801 ]

James Dumay made changes - 2016-06-15 05:19

Sprint

1.0-m8 [ 46 ]

James Dumay made changes - 2016-06-15 05:19

Rank

Ranked lower

James Dumay made changes - 2016-06-20 01:17

Sprint

1.0-japan-m9 [ 46 ]

1.0-japan-m9, 1.0-m10 [ 46, 66 ]

James Dumay made changes - 2016-06-27 00:39

Sprint

1.0-japan-m9, 1.0-m10 [ 46, 66 ]

1.0-japan-m9 [ 46 ]

James Dumay made changes - 2016-06-27 00:39

Sprint

1.0-japan-m9 [ 46 ]

1.0-japan-m9, 1.0-m11 [ 46, 71 ]

James Dumay made changes - 2016-06-27 00:39

Rank

Ranked higher

James Dumay made changes - 2016-06-27 02:40

Rank

Ranked higher

James Dumay made changes - 2016-06-27 02:40

Rank

Ranked higher

James Dumay made changes - 2016-07-04 00:44

Sprint

1.0-japan-m9, 1.0-m11 [ 46, 71 ]

1.0-japan-m9, 1.0-m12 [ 46, 76 ]

James Dumay made changes - 2016-07-04 00:44

Rank

Ranked lower

James Dumay made changes - 2016-07-13 03:16

Sprint

1.0-japan-m9, 1.0-m12 [ 46, 76 ]

1.0-japan-m9 [ 46 ]

James Dumay made changes - 2016-07-13 03:16

Rank

Ranked lower

James Dumay made changes - 2016-07-13 04:05

Sprint

1.0-japan-m9 [ 46 ]

1.0-japan-m9, 1.0-m12 [ 46, 76 ]

James Dumay made changes - 2016-07-13 04:05

Rank

Ranked lower

James Dumay made changes - 2016-07-13 04:06

Priority

Minor [ 4 ]

Critical [ 2 ]

James Dumay made changes - 2016-07-18 16:51

Sprint

1.0-japan-m9, 1.0-m12 [ 46, 76 ]

1.0-japan-m9 [ 46 ]

James Dumay made changes - 2016-07-18 16:51

Rank

Ranked higher

James Dumay made changes - 2016-07-18 16:51

Sprint

1.0-japan-m9 [ 46 ]

1.0-japan-m9, 1.0-m11 [ 46, 71 ]

James Dumay made changes - 2016-07-18 16:51

Rank

Ranked lower

James Dumay made changes - 2016-07-18 16:54

Sprint

1.0-japan-m9, 1.0-m11 [ 46, 71 ]

1.0-japan-m9, 1.0-m11, 1.0-m12 [ 46, 71, 76 ]

R. Tyler Croy made changes - 2016-07-25 23:52

Workflow

JNJira [ 171373 ]

JNJira + In-Review [ 184278 ]

James Dumay made changes - 2016-08-01 00:45

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12 [ 46, 71, 76 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-beta-1 [ 46, 71, 76, 86 ]

James Dumay made changes - 2016-08-16 21:35

Priority

Critical [ 2 ]

Major [ 3 ]

James Dumay made changes - 2016-08-22 01:13

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1 [ 46, 71, 76, 86 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, 1.0-beta-1 [ 46, 71, 76, 86, 96 ]

James Dumay made changes - 2016-08-22 11:01

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, 1.0-beta-1 [ 46, 71, 76, 86, 96 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1 [ 46, 71, 76, 86 ]

James Dumay made changes - 2016-08-22 11:01

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1 [ 46, 71, 76, 86 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, 1.0-beta-2 [ 46, 71, 76, 86, 101 ]

James Dumay made changes - 2016-08-22 11:01

Rank

Ranked higher

James Dumay made changes - 2016-08-29 00:51

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, 1.0-beta-4 [ 46, 71, 76, 86, 101 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, 1.0-beta-2 [ 46, 71, 76, 86, 111 ]

James Dumay made changes - 2016-08-29 00:51

Rank

Ranked lower

James Dumay made changes - 2016-09-02 00:04

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, 1.0-beta-2 [ 46, 71, 76, 86, 111 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1 [ 46, 71, 76, 86 ]

James Dumay made changes - 2016-09-02 00:04

Rank

Ranked lower

James Dumay made changes - 2016-09-05 01:34

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1 [ 46, 71, 76, 86 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, 1.0-beta-3 [ 46, 71, 76, 86, 101 ]

James Dumay made changes - 2016-09-05 01:34

Rank

Ranked lower

James Dumay made changes - 2016-09-23 05:52

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, 1.0-b07/b08 [ 46, 71, 76, 86, 101 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, 1.0-beta-5 [ 46, 71, 76, 86, 106 ]

James Dumay made changes - 2016-09-23 05:52

Rank

Ranked lower

James Dumay made changes - 2016-10-24 02:43

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic [ 46, 71, 76, 86, 106 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, indian [ 46, 71, 76, 86, 106, 126 ]

Tom FENNELLY made changes - 2016-10-28 10:05

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, indian [ 46, 71, 76, 86, 106, 126 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic [ 46, 71, 76, 86, 106 ]

James Dumay made changes - 2016-10-31 01:08

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic [ 46, 71, 76, 86, 106 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, arctic [ 46, 71, 76, 86, 106, 131 ]

James Dumay made changes - 2016-10-31 01:08

Rank

Ranked lower

James Dumay made changes - 2016-11-07 00:01

Rank

Ranked higher

James Dumay added a comment - 2016-11-07 00:02

tfennelly this one has been hanging around for quite a few sprints. Is it still valid?

James Dumay added a comment - 2016-11-07 00:02 tfennelly this one has been hanging around for quite a few sprints. Is it still valid?

James Dumay made changes - 2016-11-09 04:49

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, arctic [ 46, 71, 76, 86, 106, 131 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic [ 46, 71, 76, 86, 106 ]

James Dumay made changes - 2016-11-09 04:49

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic [ 46, 71, 76, 86, 106 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, tasman [ 46, 71, 76, 86, 106, 136 ]

James Dumay made changes - 2016-11-09 04:49

Rank

Ranked higher

James Dumay made changes - 2016-11-14 02:00

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, tasman [ 46, 71, 76, 86, 106, 136 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic [ 46, 71, 76, 86, 106 ]

James Dumay made changes - 2016-11-14 02:00

Rank

Ranked lower

James Dumay made changes - 2016-12-06 00:16

Component/s

blueocean-plugin [ 21481 ]

James Dumay made changes - 2017-01-13 02:34

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic [ 46, 71, 76, 86, 106 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, pannonian [ 46, 71, 76, 86, 106, 211 ]

James Dumay made changes - 2017-01-13 02:34

Rank

Ranked lower

James Dumay made changes - 2017-01-13 02:55

Rank

Ranked higher

James Dumay made changes - 2017-01-13 02:56

Priority

Major [ 3 ]

Minor [ 4 ]

James Dumay made changes - 2017-01-13 05:49

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, pannonian [ 46, 71, 76, 86, 106, 211 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, panthalassa [ 46, 71, 76, 86, 106, 221 ]

James Dumay made changes - 2017-01-13 05:49

Rank

Ranked lower

James Dumay made changes - 2017-01-13 05:49

Rank

Ranked higher

James Dumay made changes - 2017-01-19 23:48

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, panthalassa [ 46, 71, 76, 86, 106, 221 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, pannonian [ 46, 71, 76, 86, 106, 211 ]

James Dumay made changes - 2017-01-19 23:48

Rank

Ranked lower

James Dumay made changes - 2017-02-04 08:13

Sprint

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, pannonian [ 46, 71, 76, 86, 106, 211 ]

1.0-japan-m9, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, atlantic, pannonian, iapetus [ 46, 71, 76, 86, 106, 211, 216 ]

James Dumay made changes - 2017-02-04 08:15

Rank

Ranked higher

James Dumay made changes - 2017-02-04 08:18

Rank

Ranked lower

Tom FENNELLY made changes - 2017-02-14 19:17

Status

Open [ 1 ]

In Progress [ 3 ]

Tom FENNELLY added a comment - 2017-02-15 10:02

This is no longer an issue. I'm guessing we made some other changes at some point that made this go away. We are caching a ref to the deleted object, allowing the permission check to happen. Not entirely sure other than those events are going through fine now.

Tom FENNELLY added a comment - 2017-02-15 10:02 This is no longer an issue. I'm guessing we made some other changes at some point that made this go away. We are caching a ref to the deleted object, allowing the permission check to happen. Not entirely sure other than those events are going through fine now.

Tom FENNELLY made changes - 2017-02-15 10:02

Resolution		Cannot Reproduce [ 5 ]
Status	In Progress [ 3 ]	Closed [ 6 ]

People

Assignee:: Tom FENNELLY

Reporter:: Tom FENNELLY

Votes:: 0 Vote for this issue

Watchers:: 2 Start watching this issue

Dates

Created:: 2016-05-26 14:23

Updated:: 2017-02-15 10:02

Resolved:: 2017-02-15 10:02