[jira] [Comment Edited] (KAFKA-18919) Clarify that KafkaPrincipalBuilder classes must also implement KafkaPrincipalSerde

["Ismael Juma (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/KAFKA-18919?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17932685#comment-17932685 ] 

Ismael Juma edited comment on KAFKA-18919 at 3/5/25 4:19 PM:
-------------------------------------------------------------

Perhaps. I think it would be a good idea to have a KIP where we can make sure we are not missing any important details. If we agree that the compatibility risk is indeed acceptable for a minor/feature release, we can perhaps do it in 4.1.


was (Author: ijuma):
Perhaps. I think it would be a good idea to have a KIP where we can make sure we are not missing any important details. If we agree that the compatibility risk is indeed acceptable for a feature release, we can perhaps do it in 4.1.

> Clarify that KafkaPrincipalBuilder classes must also implement KafkaPrincipalSerde
> ----------------------------------------------------------------------------------
>
>                 Key: KAFKA-18919
>                 URL: https://issues.apache.org/jira/browse/KAFKA-18919
>             Project: Kafka
>          Issue Type: Sub-task
>            Reporter: Mickael Maison
>            Assignee: Szu-Yung Wang
>            Priority: Major
>             Fix For: 4.0.0
>
>
> In KRaft, custom KafkaPrincipalBuilder implementations must also implement KafkaPrincipalSerde, otherwise brokers are not able to forward requests to the controller.
> So we need to update our docs.
> -We should also be able to do some cleanup in the code. For example RequestContext uses Optional<KafkaPrincipalSerde>, we should be able to remove Optional since it's now always present.- moved to KAFKA-18922



--
This message was sent by Atlassian Jira
(v8.20.10#820010)