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.

Sign up for GitHub

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

Jump to bottom

[Branch-2.7]Forbid to read other topic's data in managedLedger layer #11913

Merged
merged 2 commits into from
Sep 23, 2021
Merged

[Branch-2.7]Forbid to read other topic's data in managedLedger layer #11913

merged 2 commits into from
Sep 23, 2021

Conversation

hangc0276
Copy link
Contributor

Motivation

Related to #11852 #11894 #11814

If we use another topic to find the message, it will return the message

Modification

  1. Check the requested ledgerId whether belongs to the current topic before execute read operation in managedLedger layer.

@hangc0276 hangc0276 self-assigned this Sep 3, 2021
@eolivelli eolivelli merged commit 4a2ca66 into apache:branch-2.7 Sep 23, 2021
@michaeljmarshall michaeljmarshall mentioned this pull request Dec 11, 2021
Merged
@codelipenghui codelipenghui added the cherry-picked/branch-2.7 Archived: 2.7 is end of life label Dec 11, 2021
@codelipenghui codelipenghui mentioned this pull request Dec 11, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@merlimat merlimat merlimat approved these changes

@eolivelli eolivelli eolivelli approved these changes

@codelipenghui codelipenghui Awaiting requested review from codelipenghui

@lhotari lhotari Awaiting requested review from lhotari

Assignees

@hangc0276 hangc0276

Labels
area/security cherry-picked/branch-2.7 Archived: 2.7 is end of life release/2.7.4
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@hangc0276 @merlimat @eolivelli @codelipenghui