Page MenuHomePhabricator
Differential D15499

Only require view permissions for read-only Git LFS requests
ClosedPublic
Actions

Authored by epriestley on Mar 19 2016, 6:11 PM.
Tags
None
Referenced Files
F15528834: D15499.id37361.diff
Tue, Apr 22, 2:18 PM
F15518247: D15499.id37360.diff
Sat, Apr 19, 12:21 PM
F15516992: D15499.id.diff
Fri, Apr 18, 10:53 PM
F15513163: D15499.diff
Thu, Apr 17, 4:58 PM
F15468495: D15499.id37362.diff
Fri, Apr 4, 1:03 AM
F15446257: D15499.diff
Thu, Mar 27, 6:01 PM
F15419319: D15499.diff
Mar 21 2025, 5:16 AM
F15415697: D15499.id37362.diff
Mar 20 2025, 7:30 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T7789: Support Git Large File Storage
Commits
Restricted Diffusion Commit
rP76f07ec80bde: Only require view permissions for read-only Git LFS requests
Summary

Ref T7789. Implement proper detection for read-only requests. Previously, we assumed every request was read/write and required lots of permissions, but we don't need "Can Push" permission if you're only cloning/fetching/pulling.

Test Plan
  • Set push policy to "no one".
  • Fetched, got clean data out of LFS.
  • Tried to push, got useful error.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 37360.Mar 19 2016, 6:11 PM
epriestley retitled this revision from to Only require view permissions for read-only Git LFS requests.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T7789: Support Git Large File Storage.
Herald edited edge metadata. · View Herald TranscriptMar 19 2016, 6:11 PM
epriestley updated this revision to Diff 37361.Mar 19 2016, 8:09 PM
epriestley edited edge metadata.
  • Also do set_time_limit(0).
Herald edited edge metadata. · View Herald TranscriptMar 19 2016, 8:09 PM
epriestley mentioned this in T7789: Support Git Large File Storage.Mar 19 2016, 8:20 PM
chad accepted this revision.Mar 19 2016, 9:22 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Mar 19 2016, 9:22 PM
Closed by commit rP76f07ec80bde: Only require view permissions for read-only Git LFS requests (authored by epriestley, committed by epriestley). · Explain WhyMar 19 2016, 9:23 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
controller/
63 lines

Diff 37362

View Options

src/applications/diffusion/controller/DiffusionServeController.php

Loading...
Phabricator · Built by Phacility