Skip to content

[To dev/1.3] Fix subscription topic authorization bypass (#18418) - #18435

Open
Caideyipi wants to merge 1 commit into
apache:dev/1.3from
Caideyipi:cp-bc5b6a2-dev-1.3
Open

[To dev/1.3] Fix subscription topic authorization bypass (#18418)#18435
Caideyipi wants to merge 1 commit into
apache:dev/1.3from
Caideyipi:cp-bc5b6a2-dev-1.3

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Summary

  • bind subscription authorization to the authenticated RPC session user instead of the client-provided consumer username
  • enforce READ_DATA for tree-model topics before subscribe, heartbeat, and polling operations
  • recheck authorization on each operation so revoked privileges take effect immediately
  • backport of Fix subscription topic authorization bypass #18418 (bc5b6a2d162fa1f38c0553fe177430629bf55262), adapted to the tree-model APIs in dev/1.3

Verification

  • mvn -o compile -pl iotdb-core/datanode -DskipTests -Dcheckstyle.skip=true
  • mvn -o checkstyle:check -pl iotdb-core/datanode
  • mvn spotless:check -pl iotdb-core/datanode
  • git diff --check

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.

1 participant