jaykay12 commented on code in PR #5204:
URL: https://github.com/apache/solr/pull/5204#discussion_r4374306286
##########
solr/packaging/test/test_basic_auth.bats:
##########
@@ -25,7 +25,7 @@ setup() {
# The auth command exports some system variables that are injected as basic
auth username and password,
# however that defeats our test so fake that out via --solr-include-file
param specifing a bogus path.
- solr auth enable --type basicAuth --credentials name:password
--solr-include-file /force/credentials/to/be/supplied
+ SOLR_INCLUDE=/force/credentials/to/be/supplied solr auth enable --type
basicAuth --credentials name:password
Review Comment:
not fully sure about this aspect. will have to dig little deeper here.
--
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]
