[GitHub] [kafka] pprovenzano commented on a diff in pull request #13114: KAFKA-14084: SCRAM support in KRaft.

["pprovenzano (via GitHub)" <gi...@apache.org>]

pprovenzano commented on code in PR #13114:
URL: https://github.com/apache/kafka/pull/13114#discussion_r1100451315


##########
core/src/main/scala/kafka/server/ControllerApis.scala:
##########
@@ -816,6 +817,17 @@ class ControllerApis(val requestChannel: RequestChannel,
       }
   }
 
+  def handleAlterUserScramCredentials(request: RequestChannel.Request): CompletableFuture[Unit] = {
+    val alterRequest = request.body[AlterUserScramCredentialsRequest]
+    val context = new ControllerRequestContext(request.context.header.data, request.context.principal,

Review Comment:
   Lets chat about this. Currently talking directly with the controller is not allowed for these operations. Only brokers are allowed.



-- 
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: jira-unsubscribe@kafka.apache.org

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