pt
/
superset
mirror of https://github.com/apache/superset.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Get 'Test Connection' button to take 'engine_params' into account (#5830)

This commit is contained in:
Maxime Beauchemin 2018-09-07 14:47:01 -07:00 committed by GitHub
parent 9f4d67b49b
commit 4a62ef3ae0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
superset/templates/superset/models/database/macros.html
View File

@ -1,7 +1,7 @@
{% macro testconn() %}
<script>
$("#sqlalchemy_uri").parent()
.append('<button id="testconn" class="btn">{{ _("Test Connection") }}</button>');
.append('<button id="testconn" class="btn btn-sm btn-primary">{{ _("Test Connection") }}</button>');
$("#testconn").click(function(e) {
e.preventDefault();
var url = "/superset/testconn";

8
superset/views/core.py
View File

@ -1701,16 +1701,16 @@ class Superset(BaseSupersetView):
username),
)
connect_args = (
engine_params = (
request.json
.get('extras', {})
.get('engine_params', {})
.get('connect_args', {}))
.get('engine_params', {}))
connect_args = engine_params.get('connect_args')
if configuration:
connect_args['configuration'] = configuration
engine = create_engine(uri, connect_args=connect_args)
engine = create_engine(uri, **engine_params)
engine.connect()
return json_success(json.dumps(engine.table_names(), indent=4))
except Exception as e: