-
Notifications
You must be signed in to change notification settings - Fork 13.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add HTTP Basic Auth for Druid (Issue #4776) #6795
Conversation
This should resolve [Issue apache#4776](apache#4776) Add HTTP Basic Auth to the Druid Broker. Signed-off-by: Don Bowman <don@agilicus.com>
Codecov Report
@@ Coverage Diff @@
## master #6795 +/- ##
==========================================
+ Coverage 63.84% 65.33% +1.49%
==========================================
Files 421 422 +1
Lines 20445 21455 +1010
Branches 2218 2314 +96
==========================================
+ Hits 13053 14018 +965
- Misses 7260 7308 +48
+ Partials 132 129 -3
Continue to review full report at Codecov.
|
The other piece of this is in druid-io/pydruid#149 (for PyDruid SQL mode) |
Signed-off-by: Don Bowman <don@agilicus.com>
93e3fe7
to
83f72ec
Compare
are there any additional changes needed? |
8172ca0
to
83f72ec
Compare
Signed-off-by: Don Bowman <don@agilicus.com>
is there anything else needed here? |
I don't really see how this travis failure ( |
This should resolve Issue #4776
Add HTTP Basic Auth to the Druid Broker.
Screenshot of change:
Signed-off-by: Don Bowman don@agilicus.com