Skip to content

Commit

Permalink
Fix: make ClickHouse password and username truly optional (#3362)
Browse files Browse the repository at this point in the history
* clickhouse optional password

* clickhouse URL and user made optional
  • Loading branch information
aidarbek authored and arikfr committed Jan 30, 2019
1 parent 13bc910 commit 0b9f575
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions redash/query_runner/clickhouse.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,13 +68,13 @@ def _get_tables(self, schema):

def _send_query(self, data, stream=False):
r = requests.post(
self.configuration['url'],
self.configuration.get('url', "http://127.0.0.1:8123"),
data=data.encode("utf-8"),
stream=stream,
timeout=self.configuration.get('timeout', 30),
params={
'user': self.configuration['user'],
'password': self.configuration['password'],
'user': self.configuration.get('user', "default"),
'password': self.configuration.get('password', ""),
'database': self.configuration['dbname']
}
)
Expand Down

0 comments on commit 0b9f575

Please sign in to comment.