Page MenuHomePhabricator

Improve encoding management in Diffusion
Closed, ResolvedPublic

Description

@staticshock reports that a file with a single latin-1 character is degrading pretty badly in Diffusion (showing a blank file instead of describing the error or recovering from it).

Broadly, if we hit a file which has an encoding which we don't expect (i.e., not the repository's configured encoding, as per T452), we should try to recover more gracefully. Anything (explicit error, attempted conversion with note) would be better than what we do now.

See also T2999, T452.

Event Timeline

epriestley triaged this task as Normal priority.Sep 3 2013, 2:01 PM
epriestley added a project: Diffusion.
epriestley added subscribers: epriestley, staticshock.
chad changed the visibility from "All Users" to "Public (No Login Required)".Jul 3 2015, 5:22 AM
eadler added a project: Restricted Project.Aug 5 2016, 5:24 PM
epriestley claimed this task.

See T13105 for browse cases (and perhaps diff cases, eventually).