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
F15456768: D19466.id46562.diff
Sun, Mar 30, 12:02 PM
F15454579: D19466.id46563.diff
Sat, Mar 29, 7:10 PM
F15453282: D19466.id46563.diff
Sat, Mar 29, 10:40 AM
F15450860: D19466.id.diff
Fri, Mar 28, 6:37 PM
F15444276: D19466.diff
Thu, Mar 27, 8:48 AM
F15437393: D19466.id46562.diff
Tue, Mar 25, 7:09 PM
F15414029: D19466.diff
Wed, Mar 19, 10:39 PM
Unknown Object (File)
Feb 21 2025, 2:42 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