Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GFYCat to GifDeliveryNetwork Fix ? #26631

Open
BestRestUp3300 opened this issue Sep 19, 2020 · 3 comments
Open

GFYCat to GifDeliveryNetwork Fix ? #26631

BestRestUp3300 opened this issue Sep 19, 2020 · 3 comments

Comments

@BestRestUp3300
Copy link

@BestRestUp3300 BestRestUp3300 commented Sep 19, 2020

Checklist

  • I'm reporting a broken site support
  • I've verified that I'm running youtube-dl version 2020.09.14
  • I've checked that all provided URLs are alive and playable in a browser
  • I've checked that all URLs and arguments with special characters are properly quoted or escaped
  • I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-a', 'List.txt']
[debug] Batch file urls: ['https://www.reddit.com/r/NSFW_GIF/comments/f7ifa0/that_jiggle_with_every_pump/']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2020.09.14
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.19041
[debug] exe versions: ffmpeg git-2020-05-04-5767a2e, ffprobe git-2020-05-04-5767a2e
[debug] Proxy map: {}
[RedditR] f7ifa0: Downloading JSON metadata
[Gfycat] firmexcitablehorsechestnutleafminer: Downloading video info
ERROR: Unable to download JSON metadata: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1sp1mtlp\build\youtube_dl\extractor\common.py", line 627, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1sp1mtlp\build\youtube_dl\YoutubeDL.py", line 2238, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

Description

Since the move from GFYCAT to Redgifs, some links now redirect to https://www.gifdeliverynetwork.com/ , Which makes the program to fail and not work anymore with that link because it redirects the GFYCat Link to gifdeliverynetwork.

@TheRealDude2
Copy link
Contributor

@TheRealDude2 TheRealDude2 commented Sep 19, 2020

For the above link I was able to customize the extractor so that it works. Then I tried a few different gfycat links from reddit. It seems to me, that at the moment only the links that are not recognized by the gfycat extractor, but redirected to the generic extractor, are working.
I rewrote the _valid_url in a test version of the extractor, that it doesn't recognize links anymore and uses the gerneric extractor instead. This seems to work so far.
If this is the case for all links, it might be best to remove the gfycat extractor completely.

@BestRestUp3300
Copy link
Author

@BestRestUp3300 BestRestUp3300 commented Sep 19, 2020

For the above link I was able to customize the extractor so that it works. Then I tried a few different gfycat links from reddit. It seems to me, that at the moment only the links that are not recognized by the gfycat extractor, but redirected to the generic extractor, are working.
I rewrote the _valid_url in a test version of the extractor, that it doesn't recognize links anymore and uses the gerneric extractor instead. This seems to work so far.
If this is the case for all links, it might be best to remove the gfycat extractor completely.

That is awesome if it works. Mind sharing a test version of it working. Would be really nice. This is also only for links that redirect from gfycat to gifdeliverynetwork. The gfycat links that open in gfycat are fine.

@TheRealDude2
Copy link
Contributor

@TheRealDude2 TheRealDude2 commented Sep 20, 2020

Can you send me a link gfycat/gfycat link that works for you?

If I have a useful trial version that works with the link above, but keeps the old function, I will post the changes. Not everything works as it should yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.