Difference between revisions of "Adding Embed Whitelist Items"

From Dreamwidth Notes
Jump to: navigation, search
(Current Whitelisted Embeds ( as of June 2018))
(blip.tv is dead and I'm tired of getting SEO spam mails about it)
 
(One intermediate revision by one other user not shown)
Line 25: Line 25:
  
 
* [http://www.bandcamp.com Bandcamp]
 
* [http://www.bandcamp.com Bandcamp]
* [http://www.blip.tv Blip.tv]
 
 
* [http://www.box.com Box]
 
* [http://www.box.com Box]
  
Line 43: Line 42:
  
 
* [http://www.flickr.com Flickr]
 
* [http://www.flickr.com Flickr]
 +
* [http://funnyordie.com Funny or Die]
  
 
* [http://www.goodreads.com GoodReads]
 
* [http://www.goodreads.com GoodReads]

Latest revision as of 19:18, 8 September 2023

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)