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

Comments

hefox’s picture

hefox commented
StatusFileSize
new 2188115-media_youtube-external_calls-1.patch1.71 KB
rajmataj’s picture

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 commented
Status: Active » Needs review

Changing status to Needs Review

recrit’s picture

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

Rolled for 3.x.

kunalkursija’s picture

kunalkursija commented
StatusFileSize
new 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.