Paths

Table of Contentst

Differential D12773

Clean up miscellaneous bot issues
ClosedPublic
Actions

Authored by epriestley on May 8 2015, 2:58 PM.

Tags

None

Referenced Files

	F18869568: D12773.id.diff
	Nov 4 2025, 12:24 PM

	F18766545: D12773.id30699.diff
	Oct 7 2025, 5:15 PM

	F18765489: D12773.diff
	Oct 7 2025, 11:35 AM

	F18746382: D12773.id.diff
	Oct 3 2025, 12:18 PM

	F18740376: D12773.diff
	Oct 2 2025, 2:24 AM

	F18565982: D12773.id.diff
	Sep 9 2025, 1:56 PM

	F18497771: D12773.diff
	Sep 4 2025, 6:13 PM

	F18467193: D12773.id.diff
	Sep 2 2025, 11:59 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T7775: PhabricatorBot calls deprecated Conduit method "diffusion.getcommits"
T6073: Bot object name handler should probably only match whole words
T4377: Long-running bots which don't make Conduit calls for 24 hours fail all future calls

Commits

Restricted Diffusion Commit
rPdd88a0e9e9ea: Clean up miscellaneous bot issues

Summary

Fixes T6073. Fixes T7775. Ref T4377. This fixes a few easy things without any compatibility-breaking changes.

Stop matching things like :D123 as a revision (particularly: IPv6 addresses).
Use "diffusion.querycommits".
Support "conduit.token".

Test Plan

Used a token:

[07:52am] epriestley: -D22 :D22 #D22 /D22
[07:52am] epriestley: D22
[07:52am] phabot-local: D22: asdbb - http://local.phacility.com/D22
[07:54am] epriestley: rHGTESTX6518697472d6395f5d1cec557148957734aa76c2
[07:54am] phabot-local: http://local.phacility.com/rHGTESTX6518697472d6395f5d1cec557148957734aa76c2

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 30699.May 8 2015, 2:58 PM

epriestley retitled this revision from to Clean up miscellaneous bot issues.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added tasks: T4377: Long-running bots which don't make Conduit calls for 24 hours fail all future calls, T6073: Bot object name handler should probably only match whole words, T7775: PhabricatorBot calls deprecated Conduit method "diffusion.getcommits".

Herald added a subscriber: epriestley. · View Herald TranscriptMay 8 2015, 2:58 PM

btrahan accepted this revision.May 8 2015, 5:53 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.May 8 2015, 5:53 PM

Closed by commit rPdd88a0e9e9ea: Clean up miscellaneous bot issues (authored by epriestley, committed by epriestley). · Explain WhyMay 8 2015, 7:20 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T4377: Long-running bots which don't make Conduit calls for 24 hours fail all future calls.Aug 23 2016, 7:02 PM

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

daemon/

bot/

PhabricatorBot.php

30 lines

handler/

PhabricatorBotObjectNameHandler.php

17 lines

Diff 30710

src/infrastructure/daemon/bot/PhabricatorBot.php

Loading...

src/infrastructure/daemon/bot/handler/PhabricatorBotObjectNameHandler.php

Loading...