Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

For push display names are interpreted as regex #6641

Closed
erikjohnston opened this issue Jan 6, 2020 · 4 comments
Closed

For push display names are interpreted as regex #6641

erikjohnston opened this issue Jan 6, 2020 · 4 comments
Assignees
Labels
z-bug (Deprecated Label) z-p2 (Deprecated Label)

Comments

@erikjohnston
Copy link
Member

Specifically, if you specify a display name of [benpa] then that will cause the contains_display_name to match against the regex (^|\W)[benpa](\W|$), which matches any event with the word "a" in it.

This is because we pass the display name straight into _glob_matches and the glob to regex conversion doesn't escape regex characters in the glob. (We also probably don't want to be using display name as a glob)

@erikjohnston erikjohnston added the z-bug (Deprecated Label) label Jan 6, 2020
@benparsons
Copy link
Member

But I want this name! I blame @uhoreg for giving the idea!

@uhoreg
Copy link
Member

uhoreg commented Jan 6, 2020

FWIW, I worked around the issue by turning off notifications on "Messages containing my displayname".

@neilisfragile neilisfragile added the z-p2 (Deprecated Label) label Jan 14, 2020
@richvdh
Copy link
Member

richvdh commented Apr 10, 2020

this is disasterous for performance.

@clokep
Copy link
Member

clokep commented Apr 16, 2020

Fixed by #7271.

@clokep clokep closed this as completed Apr 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
z-bug (Deprecated Label) z-p2 (Deprecated Label)
Projects
None yet
Development

No branches or pull requests

6 participants