这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@Austin-Olacsi
Copy link
Contributor

What does this PR do?

Adds searchmysite engine

Why is this change important?

it was requested on the old project searx: searx/searx#2817
the content is mostly personal blogs that don't show up in the results of other search engines.

How to test this PR locally?

make run
!sms android

Author's checklist

Related issues

Copy link
Member

@return42 return42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks 👍

@Bnyro
Copy link
Member

Bnyro commented Apr 28, 2024

I think that it would be more robust to use a results_xpath (you'll find enough other engines in SearXNG for reference) - it just lowers chances of getting mixed results (like the url from one result but the title of another)

@Bnyro
Copy link
Member

Bnyro commented Apr 29, 2024

I think that it would be more robust to use a results_xpath (you'll find enough other engines in SearXNG for reference) - it just lowers chances of getting mixed results (like the url from one result but the title of another)

@Austin-Olacsi I've quickly pushed my suggestions, you can find these changes here: https://github.com/searxng/searxng/compare/07bdea1d95d8c87c9c4959c4a83ab4febb54a5a6..d7693e65d271c9a4f1b8c64b0f0e4df9faa55952.

Copy link
Member

@Bnyro Bnyro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks for the PR 👍

@return42 return42 merged commit c8d0b65 into searxng:master Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants