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

Reply via email to