Babydev bait of the week: add sites to whitelist for embedding

[kaberett]

This is a threefer: if you can do it for bug 4970, you can also do it for 5011 and 4959!

Thoughts? Queries? Takers? ♥

Comments

[azurelunatic]

This is a common enough task that it should have a wiki page. I've started one, with files that Fu touched in the Goodreads embed commit:

http://wiki.dwscoalition.org/wiki/index.php/Adding_Embed_Whitelist_Items

[fu]

♥

[azurelunatic]

What else should go there?

[fu]

Hmm. I think that's pretty much it. EmbedWhitellist.pm seems pretty straightforward to me (in the sense of -- edit host_path_match, that's all you need most of the time), and so is t/embed-whitelist.t (in the sense of -- add a quick san check)

I also usually make an entry with the embed to double-check but it's not really needed because of t/embed-whitelist.t. ;)

[kaberett]

Thank you, you STAR.

[azurelunatic]

*twinkles*