Page MenuHomePhabricator
Differential D7562

Don't implement SVN over HTTP
ClosedPublic
Actions

Authored by epriestley on Nov 11 2013, 9:51 PM.
Tags
None
Referenced Files
F19533169: D7562.id.diff
Tue, Jan 20, 3:58 PM
F19531816: D7562.diff
Tue, Jan 20, 7:14 AM
F19529350: D7562.id.diff
Mon, Jan 19, 3:57 PM
F19522177: D7562.diff
Sat, Jan 17, 5:15 AM
F19519420: D7562.diff
Fri, Jan 16, 5:10 AM
F19516866: D7562.diff
Thu, Jan 15, 5:02 AM
F19506655: D7562.id17082.diff
Fri, Jan 9, 4:50 PM
F19067201: D7562.id17082.diff
Nov 30 2025, 7:12 AM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rPd4eca257742a: Don't implement SVN over HTTP
Summary

Ref T2230. As far as I can tell, getting SVN working over HTTP is incredibly complicated. It's all DAV-based and doesn't appear to have any kind of binary we can just execute and pass requests through to. Don't support it for now.

  • Disable it in the UI.
  • Make sure all the error messages are reasonable.
Test Plan

Tried to HTTP an SVN repo. Tried to clone a Git repo with SVN, got a good error message.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
controller/
23 lines
61 lines
repository/
storage/
3 lines

Diff 17082

View Options

src/applications/diffusion/controller/DiffusionRepositoryEditHostingController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionServeController.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepository.php

Loading...
Phabricator · Built by Phacility