Hi Mickael, Yes, it is a more elegant solution with minor code changes but considering the backward compatibility and impact on existing custom KafkaPrincipalBuilder implementations I thought the proposed solution is a better option.
Thanks, Raghu On Thu, Nov 2, 2023 at 7:20 AM Mickael Maison <[email protected]> wrote: > Hi Raghu, > > Thanks for the KIP. > Have you considered retrieving these values using > AuthenticationContext? For example SslAuthenticationContext could have > a getter for SslPrincipalMapper. For kerberosShortNamer we could have > a new subclass of SaslAuthenticationContext, for example > GssapiAuthenticationContext. > > Thanks, > Mickael > > On Mon, Oct 16, 2023 at 8:15 PM Manikumar <[email protected]> > wrote: > > > > Hi Raghu, > > > > Thanks for the KIP. Proposed changes look good to me. > > > > Thanks, > > Manikumar > > > > On Fri, Sep 22, 2023 at 11:44 PM Raghu B <[email protected]> wrote: > > > > > Hi everyone, > > > > > > I would like to start the discussion on the KIP-982 to Access > > > SslPrincipalMapper and kerberosShortNamer in Custom > KafkaPrincipalBuilder > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-982%3A+Access+SslPrincipalMapper+and+kerberosShortNamer+in+Custom+KafkaPrincipalBuilder > > > > > > Looking forward to your feedback! > > > > > > Thanks, > > > Raghu > > > >
