Page MenuHomePhabricator

Stashbot shouldn't subscribe itself to tasks
Closed, ResolvedPublic

Description

@Stashbot shouldn't subscribe itself to tasks - cf. @gerritbot's actions

Event Timeline

Stashbot doesn't have the bot flag, so I don't think there is any way to keep it from getting associated automatically with tasks it comments on. Out of random curiosity, what harm does it cause?

I didn't say it causes any harm, though some inconveniences are relevant - T115742: Feature request: filter task's bot CCs to the end of the ellipsis. It's for the sake of consistency as well.

From https://secure.phabricator.com/book/phabricator/article/users/#bot-accounts

The Bot role for an account can not be changed after the account is created. This prevents administrators form changing a normal user into a bot, retrieving their Conduit certificate, and then changing them back (which would allow administrators to gain other users' credentials).

Since there is no way to covert the Stashbot account to a bot account, I'm going to decline this request.

Reopening for a bit until it's verified that the manual doesn't mean only there is no straightforward way via web UI (ie. is it or is it not changeable via direct db access?)

bd808 triaged this task as Low priority.May 20 2016, 2:31 AM
bd808 edited projects, added Tools, Phabricator; removed User-bd808.

@Danny_B: stashbot is now marked as a bot. I'm not sure that will prevent it from being subscribed to tasks though.

Thanks. @bd808 mentioned necessity of botflag, so he perhaps knows, what and how to set it...

Mentioned in SAL [2016-06-03T17:24:31Z] <bd808> Check to see if having bot flag in phab fixes autosubscribe for T135790

bd808 claimed this task.

It looks to me like setting the bot flag in the Phab db has stopped the autosubscribe action.

bd808 moved this task from To Do to Done on the User-bd808 board.

Phabricator seems to have changed the default behavior related to the bot account flag. @gerritbot is now getting subscribed to tasks on comment as well: T131459#2369285

@bd808 this is known upstream.

@Paladox: Any link handy? Thanks in advance!

@Aklapper https://secure.phabricator.com/T11035

It also says @mention in the comment box is expected.