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.

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

Kibana route handler is duplicating hash + query params in some scenarios #32721

Closed
jasonrhodes opened this issue Mar 7, 2019 · 3 comments · Fixed by #32732
Closed

Kibana route handler is duplicating hash + query params in some scenarios #32721

jasonrhodes opened this issue Mar 7, 2019 · 3 comments · Fixed by #32732
Labels
bug Fixes for quality problems that affect the customer experience

Comments

@jasonrhodes
Copy link
Member

Kibana version:
master at least, for now

Elasticsearch version:
8.0

Server OS version:

Browser version:
Chrome, Version 72.0.3626.121 (Official Build) (64-bit)

Browser OS version:
Mac OS X 10.14.3

Original install method (e.g. download page, yum, from source, etc.):
from source

Describe the bug:
Adding a query param with a value that typically requires URL encoding (e.g. "/") will cause the URL to update twice, once for the initial change, and then again with the entire hash + all query params URL encoded.

Example:
https://myserver.com/app/apm/#services?a=b/c

becomes

https://myserver.com/app/apm#/services?a=b%2Fc#%2Fservices%3Fa=b%252Fc

This appears to happen throughout Kibana. Confirmed in:

  • discover
  • visualize
  • dashboard
  • canvas
  • maps
  • machine learning
  • infrastructure
  • logs
  • apm
  • uptime
  • graph
  • dev tools
  • stack monitoring
  • management

ng-kibana-route-mangling

Steps to reproduce:

  1. Go to any URL in Kibana
  2. Manually add a query param to whatever URL appears, like &a=b/c, press "enter"
  3. Watch as the URL is updated to include an encoded copy of everything after the '#' in the original URL

Note: it will continue to copy everything repeatedly as you update the URL, copying the last copy in a recursive way.

Expected behavior:
The URL should only update once and not duplicate the hash + search string in an encoded copy at the end of the URL.

@jasonrhodes
Copy link
Member Author

Could possibly be related to #32372

@spalger
Copy link
Contributor

spalger commented Mar 8, 2019

@jasonrhodes @sqren would you mind trying out #32732?

@jasonrhodes jasonrhodes added the bug Fixes for quality problems that affect the customer experience label Mar 8, 2019
@jasonrhodes
Copy link
Member Author

@spalger yep those changes seem to fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants