Skip to content
This repository has been archived by the owner on Sep 7, 2023. It is now read-only.

Add RightDao search engine #2820

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eagle-dogtooth
Copy link

What does this PR do?

Adds RightDao search engine to settings.yml.

Why is this change important?

To add additional search choices to Searx. This is a fairly new general search engine with its own independent database and claims a no-tracking policy.

How to test this PR locally?

Author's checklist

Passes both make run and make test.pylint

I've been successfully including RightDao search results with no errors for several days with this code added to my own Searx instance.

Related issues

Add the RightDao search engine to settings.yml
@eagle-dogtooth
Copy link
Author

eagle-dogtooth commented May 18, 2021

Please hold off on adding this update. I found a flaw. When RightDao's search results include current news items at the top, it displaces the descriptions by that many number of items and the wrong descriptions are associated with each URL.

A tentative fix is to replace this line:
content_xpath : //div[@ class="description"]

With this:
content_xpath : //div/span[@ class="date"]

I'd like to give this a test run for a few days to see if it fixes the problem before resubmitting the PR.

@unixfox unixfox marked this pull request as draft May 18, 2021 22:33
@unixfox
Copy link
Member

unixfox commented May 18, 2021

Undraft this PR when it's ready.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants