Skip to content

PDFs no longer open reliably on Chrome #8116

Closed

Description

@ara4n

Assume this is a regression caused by matrix-org/synapse#4284 :/

Activity

  1. lampholder commented on Jan 21, 2019

    @lampholder
    Member

    We thought we'd tested to verify that this wasn't the case, but obviously not.

    Now as I understand it it's basically a trade off - if we want .pdfs to open on Chrome, we need to run the risk of firefox being vulnerable to running js code that's been uploaded to the media repo.

  2. added
    S-MinorImpairs non-critical functionality or suitable workarounds exist
    on Jan 21, 2019
  3. lampholder commented on Jan 21, 2019

    @lampholder
    Member

    We could look into mitigating this by setting the download content disposition on pdfs, perhaps from within Synapse, perhaps from Riot.

  4. jryans commented on Jan 21, 2019

    @jryans
    Collaborator

    I guess another option would be to tailor the CSP based on browser UA, but I know that's pretty gross and frowned upon.

  5. turt2live commented on Feb 13, 2019

    @turt2live
    Member

    Closing in favour of #8605 (higher priority)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2S-MinorImpairs non-critical functionality or suitable workarounds existT-Defect

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions