Fix issue reported from github
ClosedPublic
Actions

Authored by btrahan on Oct 10 2013, 10:05 PM.

Tags

None

Referenced Files

	F18641155: D7284.id.diff
	Thu, Sep 18, 7:49 PM

	F18630953: D7284.diff
	Tue, Sep 16, 1:46 PM

	F18627535: D7284.diff
	Tue, Sep 16, 4:15 AM

	F18193339: D7284.id16398.diff
	Aug 17 2025, 5:18 AM

	F18172993: D7284.id.diff
	Aug 15 2025, 7:21 PM

	F18114515: D7284.diff
	Aug 13 2025, 4:46 AM

	F17699279: D7284.id.diff
	Jul 15 2025, 7:52 PM

	Unknown Object (File)
	Jun 1 2025, 10:26 PM

Subscribers

Details

Reviewers

epriestley

Commits

Restricted Diffusion Commit
rPdb71bf6128c8: Fix issue reported from github

Summary

we filter the $actors above such that its possible to have no $actor anymore (if $actor is not a deliverable email address). ergo, make sure we have actor before we start calling methods.

Fixes github issue 403

Test Plan

logic on this one - not 100% sure how to easily reproduce

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

Ah, thanks. I took a peek at this and imagined writing a similar patch, but then forgot about it instead. Pretty sure this is the way to go, though.

Closed by commit rPdb71bf6128c8 (authored by @btrahan).

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

storage/

PhabricatorMetaMTAMail.php

11 lines

Diff 16399

View Options

src/applications/metamta/storage/PhabricatorMetaMTAMail.php

Fix issue reported from githubClosedPublicActions