Ref T10405. If gd is not installed, a number of unit tests currently fail because they generate synthetic users and try to composite profile pictures for them.
Instead, just fall back to a static default if gd is not available.
Details
Details
Faked this pathway, created a new user, saw a default profile picture.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- nogd
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 18906 Build 25495: Run Core Tests Build 25494: arc lint + arc unit
Event Timeline
Comment Actions
I see a handful of other places where we use imagecreatefromstring(); should we be fixing those as well?
Comment Actions
Ideally, yeah -- I merged everything I could remember/find into T10405 as a sort of umbrella for these. In this case, Lyuba was just hitting a narrow issue with getting arc diff to make it through the workflow.
These can be kind of a pain to hunt down and not having gd is rare, and some of them already get guarded in other ways, and in other cases we're probably reasonable already (we can't really do anything about actual thumbnailing, for example, I think) or the issue is entangled with other stuff (T13016 / T5258). So I didn't go on a hunting expedition, but it's definitely desirable that we're more robust against gd being missing than we currently are.