-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Smart Search with highlight induce Error code 414 Bad URI - Too Long #28239
Comments
Is this issue related to function getTokenData? Perhaps this function should limit the number of returned results? |
Is this still an issue? In J4 this should be fixed. I can't replicate this. |
Closing as it seems to be fixed in j4. @gbonnefille if the problem still exist for you, please comment and we can re-open the issue. |
I have the same issue on Joomla 4 (using latest 4.2.7 version). Request-URI Too Long It doesn't do with all the search results, but seems only with some articles. If i increase the max URI lenght using the following directive in the Apache.conf it doesn't give the error but the browser remain loading the article: The format of the URL is: Update: It seems related to the term searched. |
Hi, i have the same problem in joomla 4.3.1 |
The highlighting code adds all matching terms to the URL, which can be a lot of matching terms, especially in languages like german. Maybe we have to add an upper limit to the number of terms to highlight. |
I wrote a PR that should fix this: #41463 Please test. Since we have a PR, I'm closing this one. :-) |
Steps to reproduce the issue
Smart Search highlighting activated.
Try to search "document d'organisation"
Click on first result.
Expected result
Have the document displayed.
Actual result
Apache 414 Error : Bad URI - Too Long
System information (as much as possible)
Joomla 3.9.15.
Additional comments
I reproduce the problem with a simpler "document organisation d" (D alone).
While investigating, I found that the whole generated URI is 4096 bytes.
Decoding the highlight parameter (base64 decode) I found that Smart Search added many terms starting with "d'" (D quote).
The text was updated successfully, but these errors were encountered: