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

Error when attempting catalog search #176

Closed
KatharineV opened this issue Apr 12, 2024 · 6 comments · Fixed by samvera/hyku#2190
Closed

Error when attempting catalog search #176

KatharineV opened this issue Apr 12, 2024 · 6 comments · Fixed by samvera/hyku#2190
Labels
bug something isn't working Knapsack Upgrade needs rework issue needs additional work

Comments

@KatharineV
Copy link

Since the Knapsack upgrade, about 3/4 of the time I enter a term in the catalog search box on production (either tenant, and from any location--home page, a work page, a collection page, etc.), the site fails to load and I get a "Sorry, something went wrong" error message. Below are screenshots on ADL prod of a search from the homepage and from a work page.

Image

https://adl.b2.adventistdigitallibrary.org/catalog?utf8=%E2%9C%93&locale=en&search_field=all_fields&q=Spectrum
Image

Image

https://adl.b2.adventistdigitallibrary.org/catalog?utf8=%E2%9C%93&parent_query=The+First+Report+of+the+General+Conference+of+Christians+Expecting+the+Advent+of+the+Lord+Jesus+Christ%2C+Held+in+Boston%2C+Oct.+14%2C+15%2C+1840&search_field=all_fields&q=cardinal
Image

SDAPI example, to show it's happening on multiple tenants:

https://sdapi.b2.adventistdigitallibrary.org/catalog?utf8=%E2%9C%93&locale=en&search_field=all_fields&q=spectrum
Image

@KatharineV KatharineV added bug something isn't working Knapsack Upgrade labels Apr 12, 2024
@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Apr 13, 2024

Confirmed I am seeing this issue with the link provided. There are no sentry errors logged due to rate limits.

adventist-knapsack-production-hyrax-57856cd456-tv97m_hyrax.log

ERROR seen in rancher logs

ActionView::Template::Error (undefined method `any_highlighting?' for #<Hash:0x00007eff34d812a8>):

This method should come from iiif print

perhaps it's because we're using include instead of prepend here.

Error seen locally:
Image

No code change was needed after we upgraded the app's iiif print (work done in another ticket).

@ShanaLMoore
Copy link
Contributor

QA: PASS ✅

Searched for the word "minute" returned results.

Image

Searched for "spectrum" correctly did not return results (because the test tenant doesn't have this data):

Image

Empty search:

Image

@KatharineV
Copy link
Author

Friends, I still get the "We're sorry but something went wrong" error message about 1/3rd of the time when I attempt a catalog search. :( I can't always replicate the behavior, but it happens seemingly at random during attempted searches of all kinds. Screenshots below show one example search URL that failed to load in multiple tests.

I can confirm the catalog search is erroring on ADL production. It errors in multiple browsers (tested Edge, Chrome, and Firefox). It errors when I am logged in and when I am logged out.

Prod search, logged in, using Firefox. Term was "medical." I searched from the homepage and from the search bar on a work page and got the same error message result.

Image

Prod search, logged out, using Edge. Term was "medical."

Image

This term has consistently failed to return a result. Three times in a row, I've reloaded the ADL homepage, searched "medical," and gotten the error message. I hope this information is helpful.

@ShanaLMoore ShanaLMoore added the needs rework issue needs additional work label Apr 18, 2024
@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Apr 18, 2024

@KatharineV I understand your frustration. Perhaps our demo caught it on a "good" time since it appears to be intermittent. I'm able to consistently recreate this issue too, with the term "medical", so thanks for finding that example. Moving this ticket back for rework.

Sentry error

Image

@orangewolf
Copy link
Contributor

@KatharineV the "Something went wrong" pages on these searches should all be resolved now. Certain types of documents were triggering this if they appeared in the results. So some results had the error where others did not. I am fairly confident that I've now covered all the different kinds of results (different work types, collections, etc) and we won't see this specific error again.

@KatharineV
Copy link
Author

I tested the three search terms that used to throw an error message on ADL prod (Spectrum, cardinal, & medical). The catalog search worked for all three terms and returned results as expected. Thank you!

ShanaLMoore referenced this issue in samvera/hyku Apr 23, 2024
t wants a solr document. lets be consistant about what we give it

This change parallels one from Adventist.

fixes scientist-softserv/adventist-dl#750

ref:
- #2190
@kirkkwang kirkkwang transferred this issue from scientist-softserv/adventist-dl May 10, 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 Knapsack Upgrade needs rework issue needs additional work
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants