recaptime.dev / phorge
@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.) hq.recaptime.dev/wiki/Phorge
phorge phabricator
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

Diffusion - missed a "dont load diffusion request" in the code serving pathway

Summary: Fixes T6939.

Test Plan: From the task, visited a URI like http://code.example.com/diffusion/REPO/repo.git/info/refs?service=git-upload-pack. Before the patch, I got an error and post patch I get a nice login prompt to provide credentials to the repository, as expected based on my confguration

Reviewers: chad, epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T6939

Differential Revision: https://secure.phabricator.com/D11348

Bob Trahan 2d904dfd 9f4a3226

+4
+4
src/applications/diffusion/controller/DiffusionServeController.php
··· 2 2 3 3 final class DiffusionServeController extends DiffusionController { 4 4 5 + protected function shouldLoadDiffusionRequest() { 6 + return false; 7 + } 8 + 5 9 public static function isVCSRequest(AphrontRequest $request) { 6 10 if (!self::getCallsign($request)) { 7 11 return null;