HomePhorge

Fix production file links for some alt-domain configurations

Description

Fix production file links for some alt-domain configurations

Summary:
We sometimes call PhabricatorEnv::getProductionURI($file->getBestURI()) or
similar, but this may currently cause us to construct a URI like this:

http://domain.com/http://cdn-domain.com/file/data/xxx/yyy/name.jpg

Instead, if the provided URI has a domain already, leave it unmodified.

Test Plan: Attached a file to a task; got an email with a valid URI instead of
an invalid URI.

Reviewers: btrahan

Reviewed By: btrahan

CC: Makinde, aran, epriestley

Differential Revision: https://secure.phabricator.com/D1622

Details

Provenance
epriestleyAuthored on Feb 15 2012, 5:06 PM
themackabuPushed on Tue, Mar 25, 8:07 PM
Parents
rP4bd336cedc75: Add a "group by priority" to the homepage revision query
Branches
Unknown
Tags
Unknown

Event Timeline