dobesv commented on issue #1972: DRILL-7562: Support HTTP Basic authentication for REST API calls URL: https://github.com/apache/drill/pull/1972#issuecomment-585361718 @ihuzenko Actually I hadn't noticed these classes shared a base class, I had thought they only implemented an interface for some reason. I refactored that check up into the base class. However, I haven't figured out a way to confirm that the message is logged correctly. When I mess around with the configuration I can never actually see that message anywhere. Not that it is likely to be super important for the message to be correct, if it's not the stack trace will give all the information anyone needs. But if you know a way to test that message, let me know. I also fixed a bug I noticed while I was working on that, and resolved conflicts.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services