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

trailing slash to be accepted. #1688

Open
madeinindiadot opened this issue Sep 13, 2024 · 0 comments
Open

trailing slash to be accepted. #1688

madeinindiadot opened this issue Sep 13, 2024 · 0 comments

Comments

@madeinindiadot
Copy link

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.

Which version of Elastic are you using?

[.] elastic.v7 (for Elasticsearch 7.x)
[ ] elastic.v6 (for Elasticsearch 6.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v2 (for Elasticsearch 1.x)

Please describe the expected behavior

There should be a way to accept trailing slash - https://github.com/olivere/elastic/blob/release-branch.v7/canonicalize.go#L29

Please describe the actual behavior

https://elastic.search.test.com/es/ - My elastic URL looks like this and only return success if a trailing slash is present, since your code listed above is removing trailing slash, its timing out and give invalid response. Is there a way to customise this or fix the part where atleast one trailing slash is deemed valid

Any steps to reproduce the behavior?

setup an elastic search with a trailing slash and use this library

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant