#4644 Don't whitelist form elements in markdown processing

v1.5.0
closed
ux (103)
General
nobody
2016-06-21
2012-08-01
No

<textarea> is whitelisted, but pretty useless (and surprising) to see rendered as a real textarea. There doesn't seem to be a use for any form element to be rendered.

Our HTMLSanitizer preprocessor uses feedparser._HTMLSanitizer. We could subclass that to remove items from acceptable_elements.

It would be nice if these were automatically escaped, rather than removed.

Related

Tickets: #7571

Discussion

  • Dave Brondsema

    Dave Brondsema - 2013-07-16
    • labels: --> ux
     
  • Dave Brondsema

    Dave Brondsema - 2016-06-20
    • status: open --> in-progress
    • assigned_to: Dave Brondsema
     
  • Dave Brondsema

    Dave Brondsema - 2016-06-21

    On branch db/4644

    To test, start on the master branch and create a comment/page/ticket/anything using form tags. Then switch to this branch and that existing content (as well as any new posts) will escape the form tags.

     
  • Dave Brondsema

    Dave Brondsema - 2016-06-21
    • status: in-progress --> closed
     
  • Dave Brondsema

    Dave Brondsema - 2016-08-22
    • Milestone: unreleased --> v1.5.0
     

Log in to post a comment.