Skip to content

Update regex to capture edge cases

Carla Drago requested to merge 484896-exclude-some-strings into master

What does this MR do and why?

This adds a capture group to the MENTION_REGEX that ignores @ mentions that are within urls and email addresses.

For example:

Signed-off-by: Some Name <somename@gmail.com>
https://docs.example.com/en/some-server@3.5/admin/xxx/xxxx/xxxx

will no longer have the the occurrences of @ matched.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #484896 (closed)

Merge request reports