• 
      

    [rbslack] Fix escaping of review request summaries.

    Review Request #6570 — Created Nov. 8, 2014 and submitted — Latest diff uploaded

    Information

    rb-extension-pack
    84bc05a...

    Reviewers

    We were using Django's escape() for review request summaries, which
    covers a variety of characters. However, Slack expects that only &, <
    and > will ever be escaped, and will show any other entities as-is.

    This change adds our own escaping logic in order to not leak entities in
    the summaries.

    Posted a change with a variety of things that Django would escape. All
    characters looked fine in the link.