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 server doesn't redirect URL with trailing slashes #45528

Closed
mshustov opened this issue Sep 12, 2019 · 2 comments
Closed

Kibana server doesn't redirect URL with trailing slashes #45528

mshustov opened this issue Sep 12, 2019 · 2 comments
Labels
bug Fixes for quality problems that affect the customer experience stale Used to mark issues that were closed for being stale Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc

Comments

@mshustov
Copy link
Contributor

Kibana version: master

Steps to reproduce:

  1. run Kibana.
  2. navigate to a registered route with trailing slash at the end. for example http://localhost:5601/basePath/api/kibana/scripts/languages/ .

Expected behavior:
The server redirects to a correct route http://localhost:5601/basePath/api/kibana/scripts/languages.

Actual behavior
The server redirects to invalid route http://localhost:5601/basePath//api/kibana/scripts/languages.

Redirection logic can be broken for a long time https://github.com/restrry/kibana/blob/cdee299d0a58de9c5e98ee36a35bcd6e66265887/src/legacy/server/http/index.js#L96-L113
Don't know if it's something we want to maintain. If yes, we should cover it with integration tests.

@mshustov mshustov added bug Fixes for quality problems that affect the customer experience Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc labels Sep 12, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-platform

@Ghost---Shadow
Copy link

Can reproduce in 6.x.x and 7.x.x

@joshdover joshdover added the stale Used to mark issues that were closed for being stale label Jan 14, 2021
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 stale Used to mark issues that were closed for being stale Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc
Projects
None yet
Development

No branches or pull requests

4 participants