Page MenuHomePhabricator

Don't let users pick "whatever.git" as a repository short name, make "." work
ClosedPublic

Authored by epriestley on Nov 21 2016, 11:46 PM.
Tags
None
Referenced Files
F12835990: D16908.diff
Thu, Mar 28, 3:31 PM
F12816953: D16908.diff
Thu, Mar 28, 3:59 AM
F12815613: D16908.id40699.diff
Thu, Mar 28, 3:02 AM
F12815611: D16908.id.diff
Thu, Mar 28, 3:02 AM
Unknown Object (File)
Feb 19 2024, 2:21 AM
Unknown Object (File)
Feb 16 2024, 5:35 AM
Unknown Object (File)
Feb 7 2024, 10:52 AM
Unknown Object (File)
Jan 28 2024, 6:00 PM
Subscribers
None

Details

Summary

Fixes T11902.

  • Periods now work in short names.
  • If you try to name something ".git", no dice.
Test Plan
  • Tried to name something "quack.git", was politely rejected.
  • Named something "quack.notgit", and it worked fine.
  • Cloned Mercurial and Git repositories over SSH with ".git" and non-".git" variants without hitting any issues.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Don't let users pick "whatever.git" as a repository short name, make "." work.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 21 2016, 11:46 PM
src/applications/repository/storage/PhabricatorRepository.php
596–597

I couldn't figure out what this was talking about after some probing, so I think we're fine. SSH clones with Mercurial definitely work. Possibly we haven't seen the end of this.

This revision was automatically updated to reflect the committed changes.