Page MenuHomePhabricator
Differential D10734

Repositories - don't send emails or publish feed stories about commits from importing repositories
ClosedPublic
Actions

Authored by btrahan on Oct 21 2014, 5:02 PM.
Tags
None
Referenced Files
F18774234: D10734.id.diff
Thu, Oct 9, 5:25 PM
F18774191: D10734.diff
Thu, Oct 9, 3:46 PM
F18737709: D10734.id.diff
Wed, Oct 1, 12:11 PM
F18732696: D10734.diff
Tue, Sep 30, 7:22 PM
F18565638: D10734.id.diff
Sep 9 2025, 1:33 PM
F18465521: D10734.id25766.diff
Sep 2 2025, 7:21 AM
F18459790: D10734.id25766.diff
Sep 1 2025, 5:58 PM
F18458386: D10734.diff
Sep 1 2025, 2:22 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
chad
Maniphest Tasks
T6350: Diffusion spamming the feed/commits going weird
Commits
Restricted Diffusion Commit
rP3bf770cb30f4: Repositories - don't send emails or publish feed stories about commits from…
Summary

...also truncate authorName to 255 so that we don't get database errors. Ref T6350.

Test Plan

see T6350 - mostly doing it live - but I did sanity check and commit something and it worked!

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 25765.Oct 21 2014, 5:02 PM
btrahan added a task: T6350: Diffusion spamming the feed/commits going weird.
btrahan retitled this revision from to Repositories - don't send emails or publish feed stories about commits from importing repositories.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added reviewers: epriestley, chad.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptOct 21 2014, 5:02 PM
Harbormaster completed remote builds in B2872: Diff 25765.Oct 21 2014, 5:03 PM
chad accepted this revision.Oct 21 2014, 5:03 PM
chad edited edge metadata.

noidea

This revision is now accepted and ready to land.Oct 21 2014, 5:03 PM
Closed by commit rP3bf770cb30f4: Repositories - don't send emails or publish feed stories about commits from… (authored by btrahan, committed by btrahan). · Explain WhyOct 21 2014, 5:05 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
audit/
editor/
10 lines
repository/
storage/
7 lines
worker/
commitmessageparser/
4 lines

Diff 25766

View Options

src/applications/audit/editor/PhabricatorAuditEditor.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryCommit.php

Loading...
View Options

src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php

Loading...
Phabricator · Built by Phacility