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

Bug: google videos engine #4127

Closed
CactusPie opened this issue Dec 22, 2024 · 2 comments · Fixed by #4128
Closed

Bug: google videos engine #4127

CactusPie opened this issue Dec 22, 2024 · 2 comments · Fixed by #4128
Labels
bug Something isn't working google-engine

Comments

@CactusPie
Copy link

Version of SearXNG, commit number if you are using on master branch and stipulate if you forked SearXNG
Repository: https://github.com/searxng/searxng
Branch: master
Version: 2024.12.16+65c970bdf

How did you install SearXNG?

I installed it with docker, but it can be reproduced on public instances as well, such as the top 3 instances on https://searx.space/

What happened?

Google Videos search gives a parsing error and no videos from Google Videos are shown. Videos from other search engines appear correctly. To see the error you have to unfold the "Response time: X seconds" section

How To Reproduce

Search for any videos with Google Videos enabled and to see the error you have to unfold the "Response time: X seconds" section

Expected behavior

Google Videos results should be shown

Screenshots & Logs

image

Additional context

Technical report

Error

  • Error: searx.exceptions.SearxEngineXPathException
  • Percentage: 80
  • Parameters: ('.//div[@class="ITZIwc"]', 'index 0 not found')
  • File name: searx/engines/google_videos.py:118
  • Function: response
  • Code: c_node = eval_xpath_getindex(result, './/div[@class="ITZIwc"]', 0)
@CactusPie CactusPie added the bug Something isn't working label Dec 22, 2024
return42 added a commit to return42/searxng that referenced this issue Dec 22, 2024
return42 added a commit to return42/searxng that referenced this issue Dec 22, 2024
@return42
Copy link
Member

@CactusPie thanks for pointing out / I patched the bug in PR #4128, can you please test the PR 👍

@CactusPie
Copy link
Author

Wow, that was fast! Since it was only 2 lines, I manually put in the changes from the PR into the file and it seems to work now. Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working google-engine
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants