dobesv opened a new pull request #1972: Support HTTP Basic authentication for 
REST API calls
URL: https://github.com/apache/drill/pull/1972
 
 
   This can greatly simplify the development of HTTP clients, as well as
   use from the command line using curl/wget, since you don't have to
   deal with storing the session cookie.
   
   # [DRILL-7562](https://issues.apache.org/jira/browse/DRILL-7562): Support 
HTTP Basic authentication for REST API calls
   
   ## Description
   
   Implement support for Basic authentication.  This allows an HTTP client to 
supply the login credentials in the same request as the query.  For clients 
that do not wish to keep track of cookies, this saves an HTTP round-trip per 
query.
   
   This is useful for access via the command-line, but also for client 
libraries where storing the cookie is a nuisance.
   
   ## Documentation
   
   Supports HTTP basic authentication
   
   ## Testing
   
   I tested it using `curl -u user:pass "localhost:8049/storage.json"`.  When 
the feature was active, the request would succeed.  If not, the server returned 
a redirect to the login page.
   
   Assistance in writing a unit test would be appreciated, I didn't see one for 
the current login system to base this off and I didn't want to spend a ton of 
time trying to figure out how to write a test for this one.
   
   

----------------------------------------------------------------
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