Page MenuHomePhabricator
Create Task
Maniphest T5894

Thumbnails of uploaded files broken when not logged in
Closed, ResolvedPublic
Actions

Description

Steps to reproduce:

  1. Log out
  2. View a page with an attachment like https://secure.phabricator.com/D10285 or https://secure.phabricator.com/D10277
  3. A broken thumbnail appears, but clicking on thumbnail reveals attachment

Revisions and Commits

rP Phabricator
D10301rP94cdddc211d1 Cover redirects to files in more cases
D10299rPe8c51cd934da Fix external redirect flagging issue with image thumbnails

Related Objects

Duplicates Merged Here
T5905: Inline image rendering broken

Event Timeline

skyronic created this task.Aug 17 2014, 1:24 PM
skyronic raised the priority of this task from to Needs Triage.
skyronic updated the task description. (Show Details)
skyronic added a project: Files.
skyronic added a subscriber: skyronic.
cburroughs added a subscriber: cburroughs.Aug 18 2014, 1:02 PM
chad added subscribers: epriestley, chad.Aug 19 2014, 5:51 AM

I see lots of broken images here: https://secure.phabricator.com/T4392

pran1hiva added a commit: rPdf361470c1d9: Be more strict about "Location:" redirects.Aug 19 2014, 6:22 AM
pran1hiva removed a commit: rPdf361470c1d9: Be more strict about "Location:" redirects.
chad added a subscriber: timor.Aug 19 2014, 4:20 PM

◀ Merged tasks: T5905.

epriestley added a revision: D10299: Fix external redirect flagging issue with image thumbnails.Aug 19 2014, 9:13 PM
epriestley closed this task as Resolved.Aug 19 2014, 9:21 PM
epriestley added a commit: rPe8c51cd934da: Fix external redirect flagging issue with image thumbnails.

Closed by commit rPe8c51cd934da.

epriestley added a revision: D10301: Cover redirects to files in more cases.Aug 19 2014, 10:43 PM
epriestley added a commit: rP94cdddc211d1: Cover redirects to files in more cases.Aug 19 2014, 10:53 PM
Phabricator · Built by Phacility