Add user/pass authentication support for Druid

[Jakdaw]

What type of PR is this? (check all applicable)

• Feature

Description

Add support for user/password authentication when connecting to Druid.

[arikfr]

Thanks!

See comments.

Btw, are you using Druid or Imply?

[Jakdaw]

Good points - also I've missed the requirements bump in the PR. Will fix shortly.

[Jakdaw]

Library treats None like no parameter being passed... empty string it doesn't.... so I guess I need to go look at how self.configuration is populated.

[arikfr]

Empty strings: in another connector facing a similar issue, to make sure empty strings are ignored I did: self.configuration.get('password') or None.

[Jakdaw]

That's neat!

[arikfr]

Thanks!

[dlevison]

To overcome this issue in redash v8.0.0 you can add the username and password in the host field:
<user>:<pass>@druid-host
Not great security wise but works.

[arikfr]

@dlevison clever :)