• 
      

    Always pass text to markdown()

    Review Request #10699 — Created Sept. 7, 2019 and discarded — Latest diff uploaded

    Information

    Djblets
    master

    Reviewers

    Previously, we were passing a binary file to markdown.markdownFromFile in
    render_markdown_from_file, but this won't fly in Python 3. It expects
    a file opened with an encoding. However, Django does not let you specify
    an encoding when opening a file from storage, so that solution is out.
    We now just read the entire file into memory, decode it from UTF-8, and
    then pass that to markdown.markdown(), which returns text directly.

    With this patch applied, text file attachments render correctly
    in Review Board on Python 2.7, 3.5, 3.6, and 3.7.

    Commits

    Files