Can @group be sanitized on the Trac>Slack integration to avoid the flag

[garrett-eclipse]

Hello,

Can the content posted from core Trac to Slack sanitize the @group as it currently will flag in the #core-firehose for example;
​https://core.trac.wordpress.org/ticket/51184#comment:15
​https://core.trac.wordpress.org/ticket/50910#comment:20
​https://core.trac.wordpress.org/ticket/46431#comment:4

Screenshot coming.

Thanks

[garrett-eclipse]

@group instances of late that have flagged and can be seen highlighted in search.

[dd32]

Last time I looked into this, I was told that while it looks like it's a ping, it shouldn't actually be a ping.. I don't believe that's quite true though..

In testing, putting it inside a code tag should prevent it being a ping, but it looks like in these cases that's not helping.

Any suggestions on how we can escape it/etc that makes sense?

For what it's worth, we use Incoming webhooks for sending messages to Slack: https://meta.trac.wordpress.org/browser/sites/trunk/common/includes/slack/send.php

[dd32]

Ah, found it. ​Slack Formatting specifies that bots can't use @(channel|here|group) directly and should use <!(channel|here|group)> instead.. however.. Slack also ​has some auto-parsing which will convert it on the fly - I don't think that was always there. Since we set the link_names parameter to true, that gets triggered.

[garrett-eclipse]

Thanks for looking into this @dd32 it sounds like you've found the culprit. Reading into it sounds like we can omit the link_names param and disable the auto parsing as it only works on conversations and user groups which I don't think are desired;

We've already deprecated this functionality for user mentions, so always parsing manually will keep you prepared in case automatic parsing is deprecated for other entities like conversations or user groups in the future.

Deprecated link - ​https://api.slack.com/changelog/2017-09-the-one-about-usernames

They also mention manual parsing is preferred if they decide to deprecate other autoparsing features.

[dd32]

In 10264:

Slack: Disable link_names by default.

This probably isn't as useful today as it once was, as Slack usernames and WordPress.org usernames have diverged.

This should also prevent at-group from pinging the workspace.

Fixes #5402

[dd32]

Let's see if that worked... @group this is a test message :)

[garrett-eclipse]

Thanks @dd32 seems to work nicely.

[garrett-eclipse]

Actual success test

[ocean90]

@dd32 I think we should keep this enabled for the /here broadcasts because now the channel name is no longer linked.

Example: ​https://wordpress.slack.com/archives/C02RQBWTW/p1600279125290500

[dd32]

In 10278:

Slack: Link channel names in /here and /channel broadcast.

Props ocean90.
See #5402.