Paths

Table of Contentst

Return HTTP 500, not HTTP 200, on exception pages
ClosedPublic
Actions

Authored by epriestley on Nov 5 2013, 11:10 PM.

Tags

None

Referenced Files

	F14041872: D7507.diff
	Mon, Nov 11, 10:18 PM

	F14034230: D7507.id16928.diff
	Sat, Nov 9, 11:02 PM

	F14026303: D7507.diff
	Fri, Nov 8, 12:54 AM

	F14018225: D7507.diff
	Tue, Nov 5, 6:35 AM

	F14005661: D7507.id16931.diff
	Sun, Oct 27, 5:32 PM

	F13994040: D7507.diff
	Wed, Oct 23, 3:18 AM

	F13971228: D7507.id.diff
	Thu, Oct 17, 12:24 PM

	Unknown Object (File)
	Sep 12 2024, 6:35 AM

View All Files

Subscribers

aran

Details

Reviewers

btrahan

Maniphest Tasks

T4064: Empty password not handled correctly for Git clone

Commits

Restricted Diffusion Commit
rPd5f41ef70ed0: Return HTTP 500, not HTTP 200, on exception pages

Summary

Ref T4064. The response code here isn't normally relevant, but we can hit these via git clone http://../, etc., and it's clearly more correct to use HTTP 500.

Test Plan

Added a fake throw new Exception() and verified I got an HTTP 500 response.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

Seems good, but I'll leave @btrahan to verify.

btrahan accepted this revision.Nov 5 2013, 11:20 PM

Closed by commit rPd5f41ef70ed0 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

aphront/

configuration/

AphrontDefaultApplicationConfiguration.php

2 lines

applications/

base/

controller/

PhabricatorController.php

7 lines

Diff 16931

View Options

src/aphront/configuration/AphrontDefaultApplicationConfiguration.php