[GitHub] tysonnorris commented on issue #3482: Controller protocol should be https in controller application.conf

[GitBox <gi...@apache.org>]

tysonnorris commented on issue #3482: Controller protocol should be https in controller application.conf
URL: https://github.com/apache/incubator-openwhisk/issues/3482#issuecomment-377620702
 
 
   I don't think moving to reference.conf is the answer, since there is still a value explicitly set at https://github.com/apache/incubator-openwhisk/blob/master/tests/src/test/resources/application.conf.j2#L50, which should override it, assuming that file is rendered to the classpath before running the tests. 
   
   The log you mentioned that failed, on scans.gradle.com, how was it triggered exactly? Are you sure the tests application.conf.j2 was rendered as part of the build, with the config value you expect? 
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services