Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OF-2631: Enforce pubsub node access model #2223

Merged
merged 2 commits into from
Dec 20, 2024

Conversation

guusdk
Copy link
Member

@guusdk guusdk commented Jul 25, 2023

When items on a pubsub leaf node are processed, ensure that the access model of the node itself (and not just its collection) are taken into account.

@akrherz akrherz force-pushed the OF-2631_Pubsub-access-model-leafs branch from 9d0109f to 3754ba9 Compare September 4, 2023 18:53
@akrherz akrherz marked this pull request as draft September 4, 2023 18:54
Fishbowler
Fishbowler previously approved these changes Oct 3, 2023
@Fishbowler Fishbowler dismissed their stale review October 3, 2023 09:32

More testing ideas

@Fishbowler Fishbowler force-pushed the OF-2631_Pubsub-access-model-leafs branch from aa591a5 to 8af83fb Compare October 3, 2023 09:56
@Fishbowler
Copy link
Member

Rebased.

@Fishbowler Fishbowler force-pushed the OF-2631_Pubsub-access-model-leafs branch from 8af83fb to f1ebd5d Compare March 26, 2024 19:39
@Fishbowler
Copy link
Member

Rebased

guusdk and others added 2 commits November 1, 2024 21:40
When items on a pubsub leaf node are processed, ensure that the access model of the node itself (and not just its collection) are taken into account.
@Fishbowler Fishbowler force-pushed the OF-2631_Pubsub-access-model-leafs branch from f1ebd5d to 74e89a5 Compare November 1, 2024 21:40
@Fishbowler
Copy link
Member

Rebased

@Fishbowler
Copy link
Member

Did we address this somewhere else via other means? Or am I imagining this?

@guusdk guusdk marked this pull request as ready for review December 20, 2024 20:48
@guusdk
Copy link
Member Author

guusdk commented Dec 20, 2024

No, you didn't. I've observed the problem to exist in 5.0.0-SNAPSHOT.

@guusdk guusdk merged commit bb643e3 into igniterealtime:main Dec 20, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants