You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/06/13 02:01:36 UTC

[GitHub] [pulsar] Technoboy- opened a new pull request, #16026: [branch-2.10][fix][admin] Fix producer/consume permission can’t get schema

Technoboy- opened a new pull request, #16026:
URL: https://github.com/apache/pulsar/pull/16026

   Cherry-pick #15956.
   ### Motivation
   Currently, we need admin permissions to operate the schema API. This is because the admin permission was defined when the schema API was first added. See #1381.
   Later, then adding authentication granularity with #6428, we don't change the schema API part.  So leave the admin permission today.
   
   But the binary protocol allows the produce/consume permission to get the schema, so change the related method permission to `produce/consume`.
   
   ### Modifications
   - `get schema` need `GET_METADATA` permission
   
   ### Documentation
   
   - [x] `doc-not-needed` 
   (Please explain why)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] codelipenghui merged pull request #16026: [branch-2.10][fix][admin] Fix producer/consume permission can’t get schema

Posted by GitBox <gi...@apache.org>.

codelipenghui merged PR #16026:
URL: https://github.com/apache/pulsar/pull/16026


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org