I have no problem if this get's marked as won't fix, but need a place to host the patch.

Long story short, can't find youtube's ips to configure a bypass for a firewall, but want to still use the ability to embed as a file type (and display the embed, but not the thumbnail). I can expand on this if desired.

CommentFileSizeAuthor
#5 disabling-external-calls-2188115-5.patch1.8 KBkunalkursija
#4 2188115-media_youtube-external_calls-3x-4.patch1.76 KBrecrit
#1 2188115-media_youtube-external_calls-1.patch1.71 KBhefox
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hefox’s picture

hefox CreditAttribution: hefox commented
FileSize
2188115-media_youtube-external_calls-1.patch1.71 KB
rajmataj’s picture

rajmataj CreditAttribution: rajmataj commented

This is fantastic. A choice of whether to use a local thumbnail instead of remote thumbnails would be a welcome addition to the options of the module. I'm working in an environment where the server the Drupal instance is running on is an intranet, cut off from outside access to the web in terms of fetching outside content. When media_youtube tries to get a remote thumbnail from YouTube, it can't and kills the whole process - so the module is pretty much useless to show videos as a result.

Incorporating the ideas in this patch would be really welcomed.

visabhishek’s picture

visabhishek CreditAttribution: visabhishek as a volunteer and at Azri Solutions commented
Status: Active » Needs review

Changing status to Needs Review

recrit’s picture

recrit CreditAttribution: recrit commented
Version: 7.x-2.x-dev » 7.x-3.x-dev
FileSize
2188115-media_youtube-external_calls-3x-4.patch1.76 KB

Rolled for 3.x.

kunalkursija’s picture

kunalkursija CreditAttribution: kunalkursija at Axelerant commented
FileSize
disabling-external-calls-2188115-5.patch1.8 KB

Re-Rolling Patch, As #4 is not working for 7.x-3.x branch with Tag -> 7.x-3.7.