Page MenuHomePhabricator
Differential D18977

Fix a missing getSSHUser() callsite
ClosedPublic
Actions

Authored by epriestley on Jan 31 2018, 7:30 PM.
Tags
None
Referenced Files
F15462335: D18977.id45511.diff
Tue, Apr 1, 2:08 PM
F15453660: D18977.id45524.diff
Sat, Mar 29, 1:27 PM
F15448992: D18977.id.diff
Mar 28 2025, 8:04 AM
F15444587: D18977.diff
Mar 27 2025, 10:12 AM
F15428342: D18977.id45524.diff
Mar 23 2025, 7:44 PM
F15421158: D18977.id45511.diff
Mar 21 2025, 9:20 PM
F15401943: D18977.id.diff
Mar 17 2025, 7:52 PM
F15399930: D18977.id45511.diff
Mar 17 2025, 8:36 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Commits
rP4757ff224c7f: (stable) Fix a missing getSSHUser() callsite
rPf535981c0d61: Fix a missing getSSHUser() callsite
Summary

See https://discourse.phabricator-community.org/t/after-upgrade-git-lfs-push-ends-up-in-call-to-undefined-method-on-diffusion-git-lfs-authenticate-workflow/1047/1.

I renamed this method in D18912 but missed this callsite since the workflow doesn't live alongside the other ones.

Test Plan

Ran git push in an LFS repository over SSH. Before: fatal; after: clean push.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 31 2018, 7:30 PM
Harbormaster completed remote builds in B19300: Diff 45511.Jan 31 2018, 7:31 PM
epriestley requested review of this revision.Jan 31 2018, 7:31 PM
amckinley accepted this revision.Jan 31 2018, 8:32 PM
This revision is now accepted and ready to land.Jan 31 2018, 8:32 PM
Closed by commit rPf535981c0d61: Fix a missing getSSHUser() callsite (authored by epriestley). · Explain WhyJan 31 2018, 11:34 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
gitlfs/
2 lines

Diff 45524

View Options

src/applications/diffusion/gitlfs/DiffusionGitLFSAuthenticateWorkflow.php

Loading...
Phabricator · Built by Phacility