[fix] url shortner for long explore url (#6449)

- redirect explore url with r parameter
- redirect to relative path
This commit is contained in:
Grace Guo 2019-01-07 14:28:30 -08:00 committed by GitHub
parent 68e9d2970a
commit be1895f2dc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 2 deletions

View File

@ -761,8 +761,13 @@ class R(BaseSupersetView):
@expose('/<url_id>')
def index(self, url_id):
url = db.session.query(models.Url).filter_by(id=url_id).first()
if url:
return redirect('/' + url.url)
if url and url.url:
explore_url = '//superset/explore/?'
if url.url.startswith(explore_url):
explore_url += f'r={url_id}'
return redirect(explore_url[1:])
else:
return redirect(url.url[1:])
else:
flash('URL to nowhere...', 'danger')
return redirect('/')