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

Synapse cuts out some words from search leading to confusion #4950

Closed
gdvine opened this issue Aug 31, 2017 · 11 comments
Closed

Synapse cuts out some words from search leading to confusion #4950

gdvine opened this issue Aug 31, 2017 · 11 comments
Labels
P2 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Z-Backend Z-Synapse

Comments

@gdvine
Copy link

gdvine commented Aug 31, 2017

Description

Desktop client in OS X returns no results, ever.

Steps to reproduce

  • Click on lupe button
  • Type any search term, such as "the", click submit lupe. (Either "This Room" or "All Rooms")
  • Lupe flashes for 10 seconds or so. Below: "No results"

Expected: some results.

Log: sent

  • Desktop
  • OS X
  • Version 0.12.2
@t3chguy
Copy link
Member

t3chguy commented Aug 31, 2017

@gdvine are you in an E2E encrypted room? Searches are currently server-side and thus cannot be done on E2E messages

@turt2live
Copy link
Member

Strangely, I can reproduce this on Chrome 60 web as well:
image

The response is a 200 OK from the server:
{"search_categories":{"room_events":{"count":0,"highlights":[],"results":[]}}}

Other terms, like e and hello work, but the fails every time.

@t3chguy
Copy link
Member

t3chguy commented Aug 31, 2017

maybe Synapse cuts the and other such words out of the search phrase to get more results
because it does do quite a wide search

@turt2live
Copy link
Member

I'm going to also blame synapse. Words like it, my, and on also fail. Other stop words like must work though.

@gdvine
Copy link
Author

gdvine commented Aug 31, 2017

E2E encryption is on, so that explains why the searching fails. Perhaps a conditional should be introduced which bypasses E2E encrypted rooms, and returns a message if no unencrypted rooms are found.

Is there a proposal for client-side search anywhere?

@gdvine
Copy link
Author

gdvine commented Aug 31, 2017

Bad example – not only "the" but any string whatsoever returns no results as all my rooms are E2E.

@t3chguy
Copy link
Member

t3chguy commented Aug 31, 2017

@gdvine #2548

turning this thread into better tracking of the implicit behaviour of the and co

@t3chguy t3chguy changed the title All searches generate "No results" Synapse cuts out some words from search leading to confusion Aug 31, 2017
@lampholder
Copy link
Member

Yuck!

Whatever else we decide to do, we should tell the user what we're doing if we're modifiying their search.

How easily could we do something like:
image

@lampholder lampholder added T-Defect S-Tolerable Low/no impact on users S-Major Severely degrades major functionality or product features, with no satisfactory workaround P2 ui/ux labels Sep 5, 2017
@ara4n
Copy link
Member

ara4n commented Sep 8, 2017

This is particularly unpleasant when searching for terms in URLs: https://github.com/vector-im/riot-web/issues/4486

@grahamperrin
Copy link

Today's https://matrix.to/#/!GdmnfkDxuKZAICnOhe:matrix.org/$15054991507323727AFZiy:matrix.org ends with this string:

to be /theme and /doc :D

Seeking /theme in that room does not find that line.


Side note: I stumbled across that example, and others involving non-alphanumeric characters, whilst trying to make #5007 reproducible.

@t3chguy
Copy link
Member

t3chguy commented Apr 28, 2022

Closing in favour of matrix-org/synapse#10559

@t3chguy t3chguy closed this as completed Apr 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Z-Backend Z-Synapse
Projects
None yet
Development

No branches or pull requests

7 participants