We have moved to Github. Please open tickets there.

Opened 18 months ago

Last modified 13 months ago

#4201 new Defect

Need better parsing of wire posts for punctuation directly after usernames

Reported by: cash Owned by:
Priority: normal Milestone: Near Term Future Release
Component: The Wire Version: 1.8
Severity: minor Keywords:
Cc: brett@… Difficulty:

Description (last modified by ewinslow)

"@brett." could be the username "brett" with a period after it or it could be the username "brett."

The latter is more unlikely. A 99% solution is to not allow periods at the end of usernames in the regex for detecting them as mentions. A 100% solution is to actually check for the existence of the username. (Aside, a function to check whether a username exists would come in handy for registration.) Checking for the username also fixes cases that I see where users use the @ for displaynames.

Change History (3)

comment:1 Changed 18 months ago by Purus

This unlikely situation has happened for the user name "olajideogundipejr." :)

comment:2 Changed 14 months ago by ewinslow

  • Description modified (diff)

I don't think we should support periods at the beginning or end of usernames for this very reason.

comment:3 Changed 13 months ago by ewinslow

  • Milestone changed from Needs Review to Near Term Future Release
Note: See TracTickets for help on using tickets.