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

	F14411593: D12773.diff
	Tue, Dec 24, 11:34 AM

	Unknown Object (File)
	Fri, Dec 20, 4:32 PM

	Unknown Object (File)
	Fri, Dec 20, 4:23 PM

	Unknown Object (File)
	Mon, Dec 9, 8:44 PM

	Unknown Object (File)
	Sun, Dec 8, 1:50 PM

	Unknown Object (File)
	Sun, Dec 8, 3:17 AM

	Unknown Object (File)
	Wed, Nov 27, 5:27 PM

	Unknown Object (File)
	Sun, Nov 24, 2:33 PM

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

Branch

bot1

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 5862
Build 5882: [Placeholder Plan] Wait for 30 Seconds

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 30699

src/infrastructure/daemon/bot/PhabricatorBot.php

Loading...

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

Loading...