MLTSHP / mltshp

70%

push

buildkite

Strip `si` parameter from YouTube (#770)

* Strip `si` parameter from YouTube

This was raised as an issue (in the migration repo, presumably by
mistake); YouTube is now adding a session id parameter to share URLs
which could be used to track the originating account. This changeset
removes any such parameter from a YouTube URL when making an OEmbed.

* Removed DV Entries (#774)

Removed Dark Visitors entries, as those are now added automatically in a separate process.

* Responding to PR

Slight modification to our sourcefile tests, and we're no longer
modifying the URL with regexes to drop the `si` parameter from YouTube.

---------

Co-authored-by: Steve Cook <snarkout@gmail.com>
Co-authored-by: James Huff <macmanx2@users.noreply.github.com>
Co-authored-by: Steve Cook <steve@snarkout.org>
Co-authored-by: Brad Choate <22245+bradchoate@users.noreply.github.com>

6 of 6 new or added lines in 1 file covered. (100.0%)

4477 of 6363 relevant lines covered (70.36%)

0.7 hits per line