Page MenuHomePhabricator
Differential D19466

Prevent creation of empty repository identities
ClosedPublic
Actions

Authored by amckinley on Jun 5 2018, 10:59 PM.
Tags
None
Referenced Files
F18050019: D19466.id.diff
Sun, Aug 3, 6:11 PM
F17934573: D19466.diff
Wed, Jul 30, 10:40 PM
F17907420: D19466.id46563.diff
Tue, Jul 29, 1:17 PM
F17904951: D19466.id46564.diff
Tue, Jul 29, 11:44 AM
F17839264: D19466.id46562.diff
Sat, Jul 26, 7:46 PM
Unknown Object (File)
Sun, Jul 6, 1:34 PM
Unknown Object (File)
Jun 2 2025, 5:05 PM
Unknown Object (File)
May 6 2025, 7:08 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rP994ed5998f00: (stable) Prevent creation of empty repository identities
rPb8b2d1672dc3: Prevent creation of empty repository identities
Summary

Fixes issue reported in https://secure.phabricator.com/rPf191a66490b194785fae28c062b71be99bb14584#43240

Test Plan

Imported an SVN repo, observed clean import instead of daemon exception.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley created this revision.Jun 5 2018, 10:59 PM
Harbormaster completed remote builds in B20370: Diff 46562.Jun 5 2018, 11:00 PM
amckinley requested review of this revision.Jun 5 2018, 11:00 PM
Herald added a subscriber: Korvin. · View Herald TranscriptJun 5 2018, 11:00 PM
epriestley accepted this revision.Jun 5 2018, 11:06 PM
epriestley added inline comments.
src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php
82–83

We could probably skip this query, too, if $committer === null since we (should) never get a hit for it.

This revision is now accepted and ready to land.Jun 5 2018, 11:06 PM
amckinley added inline comments.Jun 5 2018, 11:09 PM
src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php
82–83

Yeah, I just left it so that $committer_identity would always be defined. Will clean it up.

amckinley updated this revision to Diff 46563.Jun 5 2018, 11:12 PM

Cleaner logic

Harbormaster completed remote builds in B20371: Diff 46563.Jun 5 2018, 11:13 PM
Closed by commit rPb8b2d1672dc3: Prevent creation of empty repository identities (authored by amckinley). · Explain WhyJun 5 2018, 11:14 PM
This revision was automatically updated to reflect the committed changes.
amckinley mentioned this in rPf191a66490b1: Add controllers/search/edit engine functionality to RepositoryIdentity.Jun 5 2018, 11:23 PM
epriestley added a comment.Jun 7 2018, 4:09 PM

I cherry-picked this to stable for good measure.

Revision Contents
Changeset List

PathSize
src/
applications/
repository/
worker/
commitmessageparser/
31 lines

Diff 46564

View Options

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

Loading...
Phabricator · Built by Phacility