Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add support for Druid Basic Auth to SQLAlchemy #149

Merged
merged 3 commits into from
Mar 19, 2019
Merged

Add support for Druid Basic Auth to SQLAlchemy #149

merged 3 commits into from
Mar 19, 2019

Commits on Mar 15, 2019

  1. Add support for Druid Basic Auth to SQLAlchemy 

    This adds support for basic authentication (in the same
format as the other sqlalchemy drivers)
`druid://USER:PASSWORD@localhost:8082/druid/v2/sql/`

Signed-off-by: Don Bowman <don@agilicus.com>
    @donbowman
    donbowman committed Mar 15, 2019
    Configuration menu
    Copy the full SHA
    0e9aa77 View commit details
    Browse the repository at this point in the history

  2. Switch from '' to None for user/password defaults, make auth conditional 

    Originally I had this as '' since it means the auth could always be
included. Now 'None' is used and no auth header is presented if
no user/password is specified.
    @donbowman
    donbowman committed Mar 15, 2019
    Configuration menu
    Copy the full SHA
    c57632c View commit details
    Browse the repository at this point in the history

  3. fix: make context be explicitly named to Cursor in call

    @donbowman
    donbowman committed Mar 15, 2019
    Configuration menu
    Copy the full SHA
    87f6560 View commit details
    Browse the repository at this point in the history