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

Domain whitelisting for web search results #3112

Merged
merged 14 commits into from
Jun 17, 2024
Merged

Conversation

que-nguyen
Copy link
Contributor

Note: Meta search engines like SearxNG do not support filtering results by domain natively, so this filtering is applied post-search.
Added a filter to restrict search results to specified domains.
Updated the processing of search results to apply the whitelist filter before returning the final results.
Ensured that only results from allowed domains (if specified in the whitelist) are included in the output.

@programminginhackerway
Copy link

SearxNG

Thanks, does it work well? I found that SearxNG only supports the high priority option and it can't handle the unwanted websites in results.

@que-nguyen
Copy link
Contributor Author

SearxNG

Thanks, does it work well? I found that SearxNG only supports the high priority option and it can't handle the unwanted websites in results.

Yes, it works well and addresses the issue by ensuring that only trusted websites are included and are under control.

backend/apps/rag/main.py Outdated Show resolved Hide resolved
backend/config.py Outdated Show resolved Hide resolved
backend/config.py Outdated Show resolved Hide resolved
@tjbck
Copy link
Contributor

tjbck commented Jun 17, 2024

The rest LGTM!

@tjbck
Copy link
Contributor

tjbck commented Jun 17, 2024

LGTM, Thanks!

@tjbck tjbck merged commit 20f052e into open-webui:dev Jun 17, 2024
3 of 5 checks passed
@que-nguyen que-nguyen deleted the searxng branch June 19, 2024 11:20
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.

None yet

3 participants