Skip to content

[BUG] invalidNextUrl when the value for nextUrl is the basePath (without trailing forward slash) #2097

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

Closed
cwperks opened this issue Aug 21, 2024 · 0 comments · Fixed by #2096
Labels
bug Something isn't working untriaged

Comments

@cwperks
Copy link
Member

cwperks commented Aug 21, 2024

What is the bug?

In #2048, the nextUrl validation logic was updated to account for serverBasePath. There was a miss in the logic that didn't take into account when the value for nextUrl was the basePath only. i.e. if basePath is /osd and nextUrl=%2Fosd the dashboards would give back an Invalid nextUrl parameter. error. The workaround is to add a trailing forward slash like nextUrl=%2Fosd%2F

What is the expected behavior?

If nextUrl is equal to the basePath then it should the nextUrl should be valid.

What is your host/environment?

  • OpenSearch 2.16.0
@cwperks cwperks added bug Something isn't working untriaged labels Aug 21, 2024
@cwperks cwperks changed the title [BUG] invalidNextUrl when nextUrl is only set to the basePath without trailing forward slash [BUG] invalidNextUrl when the value for nextUrl is the basePath (without trailing forward slash) Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working untriaged
Projects
None yet
1 participant