search

MarginaliaSearch / encyclopedia.marginalia.nu

Marginalia Encyclopedia
Other
10 stars 1 forks source link

Search bar not handling accented characters properly #1

Closed aitorres closed 1 year ago

aitorres commented 1 year ago

Hi, thank you for this interesting experiment! :-)

I wanted to report a bug with the search bar: if you try to search any term with an accented character (e.g. é), the application will render an 500 internal server error.

You can reproduce by typing this and clicking Search:

Screenshot 2023-08-30 at 10 39 14 PM

I see that, by doing this, the app is directed to the path https://encyclopedia.marginalia.nu/find/%E9 (note the escaped character).

Screenshot 2023-08-30 at 10 40 33 PM

If I manually set the path to /find/é, the search works as expected and results are displayed properly:

Screenshot 2023-08-30 at 10 41 00 PM

I could try finding and solving the issue but it'd take me a few days as I'd need to ramp up on Java / Spark, which I haven't used much before (unless it's a very quick change, but I'm not sure!).

And again, thanks for this project!

vlofgren commented 1 year ago

Interesting. Bug was due to a redirect not doing URL-encoding properly. Fixed now.