Difference between revisions of "Adding Embed Whitelist Items"
From Dreamwidth Notes
(→Current Whitelisted Embeds ( as of January 2017)) |
(→Current Whitelisted Embeds ( as of June 2018)) |
||
| Line 22: | Line 22: | ||
* [http://airtable.com Airtable] | * [http://airtable.com Airtable] | ||
* [http://www.archive.org Internet Archive] | * [http://www.archive.org Internet Archive] | ||
| + | * [https://audiomack.com Audiomack] | ||
* [http://www.bandcamp.com Bandcamp] | * [http://www.bandcamp.com Bandcamp] | ||
| Line 27: | Line 28: | ||
* [http://www.box.com Box] | * [http://www.box.com Box] | ||
| + | * [https://chirb.it chirb.it] | ||
* [http://cnn.com CNN] | * [http://cnn.com CNN] | ||
* [http://www.codepen.io Codepen] | * [http://www.codepen.io Codepen] | ||
| Line 64: | Line 66: | ||
* [http://www.mixcloud.com Mixcloud] | * [http://www.mixcloud.com Mixcloud] | ||
| + | * [https://mixstep.co Mixstep] | ||
* [http://www.msnbc.com MSNBC] | * [http://www.msnbc.com MSNBC] | ||
* [http://my.mail.ru my.mail.ru] | * [http://my.mail.ru my.mail.ru] | ||
Revision as of 19:55, 16 June 2018
Files you'll want to poke include:
cgi-bin/DW/Hooks/EmbedWhitelist.pm t/embed-whitelist.t
In the code hook, most sites can be added to %host_path_match with a simple regular expression to test that the URL is in the expected format. The second argument is 1 or 0 depending on whether the site supports https connections or not - you should test this.
In the test suite, you'll want to add a simple check to make sure the URL you were given as a test case passes your check to be allowed on the site. See Dev Testing for more information about setting up and running the automated tests.
Example
Add flickr embeds to our whitelist, issue #720
Add vid.me embeds, cover the other type of flickr videos
Current Whitelisted Embeds ( as of June 2018)
- Google Maps
- Google Calendar
- Google Docs (excluding forms, for security reasons)
- Google Books
