Page MenuHomePhabricator
Differential D16598

Fix a bug in the imageproxy controller
ClosedPublic
Actions

Authored by jcox on Sep 26 2016, 2:18 PM.
Tags
None
Referenced Files
F17315024: D16598.diff
Thu, Jun 26, 2:46 PM
Unknown Object (File)
May 8 2025, 12:42 AM
Unknown Object (File)
Apr 19 2025, 3:17 PM
Unknown Object (File)
Apr 16 2025, 7:35 AM
Unknown Object (File)
Apr 12 2025, 8:10 PM
Unknown Object (File)
Apr 10 2025, 6:34 PM
Unknown Object (File)
Apr 10 2025, 4:14 PM
Unknown Object (File)
Apr 9 2025, 2:00 PM
View All Files
Subscribers
epriestley
yelirekim
Tokens
"Hungry Hippo" token, awarded by epriestley."Dislike" token, awarded by jcox.

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rP7d576c3f94b5: Fix a bug in the imageproxy controller
Summary

Somehow this got through last week :( It's a bug that causes the controller to... *ahem*... just not work. Luckily nothing uses this yet so nothing was really affected.

Test Plan

Hit /file/imageproxy/?uri=http://i.imgur.com/nTvVrYN.jpg and are served a nice picture of a bird

Diff Detail

Repository
rP Phabricator
Branch
ProxyControllerTypoFix
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13839
Build 17894: Run Core Tests
Build 17893: arc lint + arc unit

Event Timeline

jcox updated this revision to Diff 39950.Sep 26 2016, 2:18 PM
jcox retitled this revision from to Fix a bug in the imageproxy controller.
jcox updated this object.
jcox edited the test plan for this revision. (Show Details)
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptSep 26 2016, 2:18 PM
Herald added subscribers: yelirekim, epriestley. · View Herald Transcript
jcox updated this object.Sep 26 2016, 2:19 PM
jcox edited the test plan for this revision. (Show Details)
jcox edited edge metadata.
jcox awarded a token.Sep 26 2016, 2:25 PM
epriestley accepted this revision.Sep 26 2016, 2:35 PM
epriestley added a reviewer: epriestley.
This revision is now accepted and ready to land.Sep 26 2016, 2:35 PM
epriestley awarded a token.Sep 26 2016, 2:35 PM
Closed by commit rP7d576c3f94b5: Fix a bug in the imageproxy controller (authored by jcox, committed by jcox). · Explain WhySep 26 2016, 2:55 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T11701: Add a lint rule for `if (CONSTANT) { ... }`.Sep 26 2016, 3:12 PM

Revision Contents
Changeset List

PathSize
src/
applications/
files/
controller/
2 lines

Diff 39950

View Options

src/applications/files/controller/PhabricatorFileImageProxyController.php

Loading...
Phabricator · Built by Phacility