Skip to content

Clean rendered content #1

@cdriehuys

Description

@cdriehuys

Right now we are trusting that users who can access the admin site to create content will not do anything malicious. The current behavior allows users to input <script> tags into their content which could lead to all sorts of XSS attacks. This issue will become especially concerning if we expand to allow user comments or anything like that.

The workaround is to use something like bleach to remove any HTML tags in the rendered output that aren't on a whitelist. Because of the extra work rendering will take, it might be smart to start storing the rendered version of a post in the database and recomputing it if the markdown content changes.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions