Fix daemon auth issue
ClosedPublic
Actions

Authored by epriestley on Oct 19 2013, 12:35 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Feb 19, 4:32 AM

	Unknown Object (File)
	Tue, Feb 11, 5:57 AM

	Unknown Object (File)
	Tue, Feb 11, 5:57 AM

	Unknown Object (File)
	Tue, Feb 11, 5:57 AM

	Unknown Object (File)
	Tue, Feb 11, 5:50 AM

	Unknown Object (File)
	Tue, Feb 11, 5:45 AM

	Unknown Object (File)
	Thu, Jan 23, 11:04 PM

	Unknown Object (File)
	Jan 17 2025, 8:28 PM

View All Files

Subscribers

aran

poop

Details

Reviewers

poop
btrahan

Commits

Restricted Diffusion Commit
rP7180199d1370: Fix daemon auth issue

Summary

I touched this code recently but it needs an unusual special case because we call through with the "omnipotent user" from the daemons. As per the TODO below, this will all get cleaned up at some point.

Test Plan

Will make @poop verify.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

Also, make the call with the omnipotent user. Broader policy controls makes parsing commits as the author fairly silly, I think. Parsing commits does not cause actions or side effects.

success!

Closed by commit rP7180199d1370 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

conduit/

call/

ConduitCall.php

3 lines

repository/

worker/

commitmessageparser/

PhabricatorRepositoryCommitMessageParserWorker.php

2 lines

Diff 16566

View Options

src/applications/conduit/call/ConduitCall.php