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

URL fragment get lost in redirect #1476

Closed
jerrywang-db opened this issue Jun 16, 2023 · 4 comments
Closed

URL fragment get lost in redirect #1476

jerrywang-db opened this issue Jun 16, 2023 · 4 comments
Labels
enhancement New feature or request triaged

Comments

@jerrywang-db
Copy link

Is your feature request related to a problem? Please describe.

When working with Kibana, the authentication expires after certain time (e.g. 30 minutes), and the page is redirected to the authentication page. And when authentication completed, the page can not be redirected back to where it was because the URL fragment is not preserved.

Describe the solution you'd like

The page should be redirected to the original URL after authentication. ElasticSearch has fixed this issue: elastic/kibana#44513

Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered.

Additional context

Add any other context or screenshots about the feature request here.

@jerrywang-db jerrywang-db added the enhancement New feature or request label Jun 16, 2023
@joshuarrrr
Copy link
Member

Thanks @jerrywang-db .

@opensearch-project/triage please transfer to https://github.com/opensearch-project/security-dashboards-plugin, as OpenSearch Dashboards itself doesn't handle authentication. If there are changes we need in core dashboards to the routing or redirected URL state, we'll likely need more specifics requirements from the security repo.

@bbarani bbarani transferred this issue from opensearch-project/OpenSearch-Dashboards Jun 19, 2023
@stephen-crawford
Copy link
Contributor

[Triage] Hi @jerrywang-db, thank you for filing this issue. @cwperks has volunteered to take a look at addressing this.

@devardee
Copy link
Contributor

devardee commented Sep 7, 2023

The requested feature is already merged #1039

@davidlago can we close this issue ?

cc: @prabhat-chaturvedi

@davidlago
Copy link

Closing, thanks for the ping! The PRs were not linked to this issue explicitly so it didn't auto-close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request triaged
Projects
None yet
Development

No branches or pull requests

6 participants