Whitelist more embeds

[trieloff]

Fix for #700 #701 and #702

For adobe/theblog#186

[codecov]

Codecov Report

Merging #703 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #703   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           60        60           
  Lines         1459      1459           
=========================================
  Hits          1459      1459           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a4d82e8...a9c1205. Read the comment docs.

[github-actions]

This PR will trigger a minor release when merged.

[kptdobe]

Ok even if it does not fully solve the Vimeo + player case (see comment in #702) (more will be required)

[rofe]

Should we add Spotify as well?

[MarquiseRosier]

I think we should just get the list from IFramely so that we don't run into the issue of not supporting embeds, unless it's something we really don't support yet @trieloff we can just fetch this resource: http://iframe.ly/domains.json

[trieloff]

I think using the iFramely whitelist is a good idea for helix pages. Let’s open another issue for that.

[MarquiseRosier]

I think using the iFramely whitelist is a good idea for helix pages. Let’s open another issue for that.

Ok! :)

[adobe-bot]

🎉 This PR is included in version 7.1.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀