Page MenuHomePhabricator

Implement "git-lfs-authenticate" over SSH
ClosedPublic

Authored by epriestley on Mar 16 2016, 2:51 PM.
Tags
None
Referenced Files
F15577127: D15482.id37308.diff
Tue, May 6, 12:27 PM
F15567698: D15482.diff
Fri, May 2, 1:31 PM
F15562373: D15482.id37330.diff
Wed, Apr 30, 2:01 AM
F15561096: D15482.id37330.diff
Tue, Apr 29, 6:07 PM
F15535202: D15482.id37330.diff
Thu, Apr 24, 3:58 AM
F15516963: D15482.id.diff
Fri, Apr 18, 10:40 PM
F15516251: D15482.id37330.diff
Fri, Apr 18, 5:31 PM
F15513587: D15482.diff
Thu, Apr 17, 8:04 PM
Subscribers
None

Details

Summary

Ref T7789. This implements a (probably) usable "git-lfs-authenticate" on top of the new temporary token infrastructure.

This won't actually do anything yet, since nothing reads the tokens.

Test Plan
$ ./bin/ssh-exec --phabricator-ssh-user admin --ssh-command 'git-lfs-authenticate'
phabricator-ssh-exec: Expected `git-lfs-authenticate <path> <operation>`, but received too few arguments.
$ ./bin/ssh-exec --phabricator-ssh-user admin --ssh-command 'git-lfs-authenticate x'
phabricator-ssh-exec: Unrecognized repository path "x". Expected a path like "/diffusion/X/" or "/diffusion/123/".
$ ./bin/ssh-exec --phabricator-ssh-user admin --ssh-command 'git-lfs-authenticate diffusion/22'
Exception: Expected `git-lfs-authenticate <path> <operation>`, but received too few arguments.
$ ./bin/ssh-exec --phabricator-ssh-user admin --ssh-command 'git-lfs-authenticate diffusion/22 y'
Exception: Git LFS operation "y" is not supported by this server.
$ ./bin/ssh-exec --phabricator-ssh-user admin --ssh-command 'git-lfs-authenticate diffusion/22 upload'
{"header":{"Authorization":"Basic QGdpdC1sZnM6NmR2bDVreWVsaXNuMmtnNXBtbnZwM3VlaWhubmI1bmI="},"href":"http:\/\/local.phacility.com\/diffusion\/22\/new-callsign-free-repository.git\/info\/lfs"}

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Implement "git-lfs-authenticate" over SSH.
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.Mar 16 2016, 6:16 PM
This revision was automatically updated to reflect the committed changes.