patrik-marton opened a new pull request, #12846:
URL: https://github.com/apache/kafka/pull/12846

   … successful login
   
   
   
   -  Added the AUTHENTICATION Priority to the JaasBasicAuthFilter. This way if 
multiple filters are present, they will be processed in the right order. 
Without this change, the basic auth filter would run after authorization 
filters.
   - Extended the filter to set the Security Context of the request in case of 
a successful login. The filters sets the user, the authentication type and and 
the isSecure property indicating whether this request was made using a secure 
channel, such as HTTPS.
   
   - Extended the JaasBasicAuthFilterTest test with a new test to see if the 
security context is set correctly
   - Previous test are working with the new change
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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.org

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

Reply via email to