Differential D11348

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

Authored by btrahan on Jan 12 2015, 4:50 PM.

Tags

None

Referenced Files

	F14394211: D11348.diff
	Sun, Dec 22, 1:46 AM

	Unknown Object (File)
	Thu, Dec 19, 12:37 AM

	Unknown Object (File)
	Wed, Dec 18, 5:05 AM

	Unknown Object (File)
	Wed, Dec 18, 1:51 AM

	Unknown Object (File)
	Fri, Dec 13, 5:55 PM

	Unknown Object (File)
	Mon, Dec 9, 8:04 AM

	Unknown Object (File)
	Thu, Dec 5, 9:45 AM

	Unknown Object (File)
	Wed, Dec 4, 4:33 PM

Subscribers

Details

Reviewers

epriestley
chad

Maniphest Tasks

T6939: InvalidArgumentException error when calling: GET /diffusion/REPO/repo.git/info/refs?service=git-upload-pack HTTP/1.1

Commits

Restricted Diffusion Commit
rP2d904dfddf44: 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

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 27258.Jan 12 2015, 4:50 PM

btrahan added a task: T6939: InvalidArgumentException error when calling: GET /diffusion/REPO/repo.git/info/refs?service=git-upload-pack HTTP/1.1.

btrahan retitled this revision from to Diffusion - missed a "dont load diffusion request" in the code serving pathway.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added reviewers: epriestley, chad.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptJan 12 2015, 4:50 PM

epriestley accepted this revision.Jan 12 2015, 4:50 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jan 12 2015, 4:50 PM

Closed by commit rP2d904dfddf44: Diffusion - missed a "dont load diffusion request" in the code serving pathway (authored by btrahan, committed by btrahan). · Explain WhyJan 12 2015, 4:50 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

controller/

DiffusionServeController.php

4 lines

Diff 27259

src/applications/diffusion/controller/DiffusionServeController.php

Loading...