Page MenuHomePhabricator

Pass instance through file transform URIs
ClosedPublic

Authored by epriestley on Feb 9 2015, 3:54 PM.
Tags
None
Referenced Files
F14042503: D11719.id.diff
Tue, Nov 12, 3:56 AM
F14002399: D11719.id28204.diff
Fri, Oct 25, 6:37 PM
F14002398: D11719.id28233.diff
Fri, Oct 25, 6:37 PM
F14000072: D11719.diff
Thu, Oct 24, 7:11 PM
F13995492: D11719.id.diff
Wed, Oct 23, 1:18 PM
F13973126: D11719.diff
Oct 17 2024, 11:15 PM
Unknown Object (File)
Oct 12 2024, 6:39 AM
Unknown Object (File)
Oct 12 2024, 6:37 AM
Subscribers

Details

Reviewers
btrahan
Commits
Restricted Diffusion Commit
rP5b1ea8c8d502: Pass instance through file transform URIs
Summary

This makes thumbnail URIs work on instanced, CDN'd installs like Phacility cluster instances.

Some of these transforms can proabably be removed, but the underlying code to generate the transform should be cleaned up too and we have some other tasks filed elsewhere about this anyway.

Test Plan

CDN'd local install now loads thumbnails properly.

Diff Detail

Repository
rP Phabricator
Branch
thumbinst
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 4397
Build 4410: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Pass instance through file transform URIs.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Feb 9 2015, 9:03 PM
This revision was automatically updated to reflect the committed changes.