maulin-vasavada commented on code in PR #3638:
URL: https://github.com/apache/cassandra/pull/3638#discussion_r1823699272
##########
examples/ssl-factory/test/unit/org/apache/cassandra/security/KubernetesSecretsPEMSslContextFactoryTest.java:
##########
@@ -114,7 +114,7 @@ public void setup()
* In order to test with real 'env' variables comment out this line
and set appropriate env variable. This is
* done to avoid having a dependency on env in the unit test.
*/
- commonConfig.put("require_client_auth", "false";
Review Comment:
Oh I recollect now. The trunk was broken for this line when I started these
changes. So I added the missing `)` to make it compile. So my local diff shows
below and the next line in the PR also shows the same. Hence I don't think
there is any issues.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
