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

Repair mailing list search #38

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

tacerus
Copy link
Member

@tacerus tacerus commented Jun 16, 2022

Hi,

Currently https://search.opensuse.org/lists/ shows two bugs for me:

  • Modern browser (Firefox) complains about the form being transmitted insecurely
  • Transmitted search queries seem to always yield zero results, even with queries like vim, which one would expect to get many results for

This patch addresses both issues using the following solutions:

  • Use https endpoint instead of plain http
  • Change query from query to q which matches the query being sent when using the search field on https://lists.opensuse.org directly

Best,
Georg

Signed-off-by: Georg Pfuetzenreuter <[email protected]>
Signed-off-by: Georg Pfuetzenreuter <[email protected]>
@hellcp
Copy link
Member

hellcp commented Jun 16, 2022

This search url is for old mailing lists software, so you would have to update the url itself as well, and also the list of mailing lists, since that requires a new id system used with mailing lists

https://lists.opensuse.org/archives/search?mlist=factory%40lists.opensuse.org&q=a is an example query of the new system, that should make it relatively easy to implement this correctly

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.

2 participants