Differential D19447

Allow number in generated clone uri
ClosedPublic
Actions

Authored by avivey on May 11 2018, 5:05 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Dec 1, 7:27 PM

	Unknown Object (File)
	Sat, Nov 30, 11:28 PM

	Unknown Object (File)
	Fri, Nov 29, 1:21 PM

	Unknown Object (File)
	Sat, Nov 23, 12:55 AM

	Unknown Object (File)
	Fri, Nov 22, 8:05 PM

	Unknown Object (File)
	Thu, Nov 21, 6:42 AM

	Unknown Object (File)
	Sun, Nov 17, 5:23 AM

	Unknown Object (File)
	Sun, Nov 10, 12:12 AM

Subscribers

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rP728130044652: Allow number in generated clone uri

Summary

See https://discourse.phabricator-community.org/t/numerical-characters-are-stripped-from-diffusion-git-repository-name-in-the-uri/

Digits are often considered reasonable characters.

Test Plan

Looked at an ascii table.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

avivey created this revision.May 11 2018, 5:05 AM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 11 2018, 5:05 AM

Herald added a subscriber: epriestley. · View Herald Transcript

Harbormaster completed remote builds in B20321: Diff 46515.May 11 2018, 5:07 AM

avivey requested review of this revision.May 11 2018, 5:07 AM

Sam2304 added a subscriber: Sam2304.May 11 2018, 9:51 AM

Thanks!

This revision is now accepted and ready to land.May 11 2018, 1:50 PM

Herald added a subscriber: Korvin. · View Herald TranscriptMay 11 2018, 1:50 PM

Closed by commit rP728130044652: Allow number in generated clone uri (authored by avivey). · Explain WhyMay 11 2018, 4:18 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

repository/

storage/

PhabricatorRepository.php

2 lines

Diff 46516

View Options

src/applications/repository/storage/PhabricatorRepository.php

Allow number in generated clone uriClosedPublicActions