Allow data URI schemes, with content type whitelist#156

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Closed

blag wants to merge9 commits intohtml5lib:masterfromblag:master

Closed

Allow data URI schemes, with content type whitelist#156

blag wants to merge9 commits intohtml5lib:masterfromblag:master

Conversation

Copy link

blag commentedMay 22, 2014

Also, use proper URIs in the tests.

Note that the content type whitelist currently only contains 'image/png'; this can be filled out in a separate PR after a more complete discussion about allowable content types.

blagand others added4 commits

April 28, 2014 10:23

Allow Data URI Schemes

aa4c3e2

Fromhttps://en.wikipedia.org/wiki/Data_URI_scheme, allow the psuedo-protocol 'data'.

Allow the data URI scheme, a whitelist for content types, and update …

93b8feb

…tests to correctly check URIs

Merge branch 'master' ofhttps://github.com/blag/html5lib-python

3b84dd5

Conflicts:html5lib/sanitizer.py

Guarantee we use an allowed protocol

39d65b5

Copy link

hoppipolla-critic-bot commentedMay 22, 2014

Critic review:https://critic.hoppipolla.co.uk/r/1612

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please donot make in-place history rewrites (e.g. viagit rebase -i orgit commit --amend) when updating this pull request.