Page MenuHomePhabricator

Call this->getRequest()->checkUrlExtension(); when doing pdf download in ElectronPdfExtension
Closed, ResolvedPublic

Description

  • As a precautionary measure, maybe call $this->getRequest()->checkUrlExtension(); when doing pdf download.

Event Timeline

Change 319340 had a related patch set uploaded (by Tobias Gritschacher):
Call checkUrlExtension() when doing pdf download

https://gerrit.wikimedia.org/r/319340

Wondering if we should introduce a new error message just for this case.. and if yes, what it should be. Opinions @Addshore @WMDE-Fisch?

Change 319340 merged by jenkins-bot:
Call checkUrlExtension() when doing pdf download

https://gerrit.wikimedia.org/r/319340

WMDE-Fisch moved this task from Currently in sprint to Done on the WMDE-TechWish board.