Re: [PR] KAFKA-18595: Remove AuthorizerUtils#sessionToRequestContext [kafka]

2025-01-21 Thread via GitHub
clolov commented on PR #18631: URL: https://github.com/apache/kafka/pull/18631#issuecomment-2605037373 Yup, sorry @dajac, I have a few changes I am tracking which I have merged in trunk and I am trying to get them in the 4.0 branch as well! -- This is an automated message from the Apache

Re: [PR] KAFKA-18595: Remove AuthorizerUtils#sessionToRequestContext [kafka]

2025-01-21 Thread via GitHub
dajac commented on PR #18631: URL: https://github.com/apache/kafka/pull/18631#issuecomment-2604797813 @clolov Don't forget to close the associated jira, thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [PR] KAFKA-18595: Remove AuthorizerUtils#sessionToRequestContext [kafka]

2025-01-21 Thread via GitHub
dajac commented on PR #18631: URL: https://github.com/apache/kafka/pull/18631#issuecomment-2604795558 Should we cherry-pick it to 4.0 branch? -- 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 s

Re: [PR] KAFKA-18595: Remove AuthorizerUtils#sessionToRequestContext [kafka]

2025-01-20 Thread via GitHub
clolov merged PR #18631: URL: https://github.com/apache/kafka/pull/18631 -- 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-unsubscr...@kafka.apache.o

[PR] KAFKA-18595: Remove AuthorizerUtils#sessionToRequestContext [kafka]

2025-01-19 Thread via GitHub
frankvicky opened a new pull request, #18631: URL: https://github.com/apache/kafka/pull/18631 JIRA: KAFKA-18595 `sessionToRequestContext` is no longer used since we are removing zk. We should remove it. ### Committer Checklist (excluded from commit message) - [ ] Verify design a