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

When using basic auth credentials in URL the query gets cut short #798

Open
OkkeKlein opened this issue Aug 25, 2023 · 4 comments · Fixed by #788
Open

When using basic auth credentials in URL the query gets cut short #798

OkkeKlein opened this issue Aug 25, 2023 · 4 comments · Fixed by #788

Comments

@OkkeKlein
Copy link
Contributor

Describe the bug
After adding credentials to the Search Engine Request Handler input, the direct Solr query returns a 404 as the URL is cut short. /serp1?q=term&fq=bs_status: does not contain the value to filter on. Nor do any of the other filters show in the request..

The request works fine without credentials.

To Reproduce
Steps to reproduce the behavior:

  1. Go to tune relevance
  2. Add credentials
  3. Click "Rerun my searches"
@epugh
Copy link
Member

epugh commented Aug 28, 2023

Once #788 is done, this will be an easy fix....

@epugh epugh linked a pull request Nov 2, 2023 that will close this issue
@epugh
Copy link
Member

epugh commented Nov 2, 2023

Need to push these changes live ;-). version 7.10

@epugh
Copy link
Member

epugh commented Dec 2, 2023

doing some testing... and so interestingly the http://reader:[email protected] approach does NOT work with the proxy... I think becuase that url pattern gets cut out.. Moving to using the explicit basic auth field is goign to fix a couple of issues I suspect ;-)

@epugh
Copy link
Member

epugh commented May 12, 2024

okay, so, I fixed this proxy thing in #1015 by actually digging it out of the proxied url, and using a proper basic auth header!

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

Successfully merging a pull request may close this issue.

2 participants