michaeljmarshall commented on code in PR #19519:
URL: https://github.com/apache/pulsar/pull/19519#discussion_r1106670604
##########
pulsar-broker/src/test/java/org/apache/pulsar/broker/service/ServerCnxTest.java:
##########
@@ -1030,6 +1030,55 @@ public void
testVerifyAuthRoleAndAuthDataFromDirectConnectionBroker() throws Exc
}));
}
+ @Test
+ public void testRefreshOriginalPrincipalWithAuthDataForwardedFromProxy()
throws Exception {
Review Comment:
I must not have written some of the assertions I thought I did. You're right
that those tests all pass. It might be worth removing the comments that
reference #19332 because your PR will make them incorrect.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
