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
F18773025: D16598.id39950.diff
Thu, Oct 9, 6:29 AM
F18754893: D16598.diff
Sun, Oct 5, 2:13 AM
F18639445: D16598.diff
Sep 17 2025, 1:57 PM
F18627615: D16598.id39950.diff
Sep 16 2025, 4:24 AM
F18514188: D16598.id39950.diff
Sep 5 2025, 10:04 AM
F18187466: D16598.id39951.diff
Aug 16 2025, 7:47 PM
F18110555: D16598.id39950.diff
Aug 11 2025, 8:58 PM
F18110171: D16598.id.diff
Aug 11 2025, 4:59 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