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

[Newgrounds] Https URLs invokes downloading from http URL. #10752

Closed
ghost opened this issue Sep 25, 2016 · 6 comments
Closed

[Newgrounds] Https URLs invokes downloading from http URL. #10752

ghost opened this issue Sep 25, 2016 · 6 comments
Labels

Comments

@ghost
Copy link

@ghost ghost commented Sep 25, 2016

  • I've verified and I assure that I'm running youtube-dl 2016.09.24

Before submitting an issue make sure you have:

  • At least skimmed through README and most notably FAQ and BUGS sections
  • Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

  • Bug report (encountered problems with youtube-dl)
  • Site support request (request for adding support for a new site)
  • Feature request (request for a new functionality)
  • Question
  • Other
$ y -v https://www.newgrounds.com/portal/view/673111
[debug] System config: []
[debug] User config: ['-4', '-f', 'best', '--prefer-free-formats', '--no-cache-dir', '--no-mtime', '--youtube-skip-dash-manifest']
[debug] Command-line args: ['-v', 'https://www.newgrounds.com/portal/view/673111']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.09.24
[debug] Python version 3.6.0b1 - Linux-4.8.0-rc5-686-i686-with-debian-stretch-sid
[debug] exe versions: ffmpeg 3.1.3, ffprobe 3.1.3, rtmpdump 2.4
[debug] Proxy map: {}
[Newgrounds] 673111: Downloading webpage
[debug] Invoking downloader on 'http://uploads.ungrounded.net/alternate/971000/971450_alternate_31675.mp4'
[download] Destination: Dancin-673111.mp4
[download] 100% of 29.73MiB in 00:02

https://www.newgrounds.com/audio/listen/549479
https://www.newgrounds.com/portal/view/673111

Downloading video/audio from https URLs should invokes downloading from https URL, otherwise, what's the point in downloading from https URLs in first place?

@ghost ghost changed the title [Newgrounds] Https URLs invoking downloading from http URL. [Newgrounds] Https URLs invokes downloading from http URL. Sep 25, 2016
@dstftw
Copy link
Collaborator

@dstftw dstftw commented Sep 25, 2016

There is absolutely no relation between the original URL's scheme and download URL's scheme. youtube-dl provides download URLs that hoster serves itself intact.

@dstftw dstftw closed this Sep 25, 2016
@dstftw dstftw added the invalid label Sep 25, 2016
@ghost
Copy link
Author

@ghost ghost commented Sep 25, 2016

@dstftw Can't it be forced to https by youtube-dl?

@dstftw
Copy link
Collaborator

@dstftw dstftw commented Sep 25, 2016

No, it does not make any sense. They may not even serve it via https.

@ghost
Copy link
Author

@ghost ghost commented Sep 25, 2016

@dstftw If it can't be done technically, then okay.

No, it does not make any sense.

Then it does not make sense to download original URL's scheme in https in first place.

They may not even serve it via https.

Yes, but usually they do. Like this hoster.

In FIrefox for example you can force it.

@dstftw
Copy link
Collaborator

@dstftw dstftw commented Sep 25, 2016

Then it does not make sense to download original URL's scheme in https in first place.

As I've already said there is no connection between that. They can provide whatever they want whichever way they want. youtube-dl just extracts what they provide no more no less.

@ytdl-org ytdl-org locked and limited conversation to collaborators Sep 25, 2016
@yan12125
Copy link
Collaborator

@yan12125 yan12125 commented Sep 25, 2016

In Firefox HTTP video/audio files are used even if the webpage is in HTTPS, so there's a warning in the address bar. It's a bug of Newgrounds, not youtube-dl.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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.