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

[NBC] Add provider support #12006

Closed
gkoelln opened this issue Feb 7, 2017 · 5 comments
Closed

[NBC] Add provider support #12006

gkoelln opened this issue Feb 7, 2017 · 5 comments
Assignees
Labels

Comments

@gkoelln
Copy link
Contributor

@gkoelln gkoelln commented Feb 7, 2017

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.02.07. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

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

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

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:

Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):

C:\WINDOWS\system32>youtube-dl http://www.nbc.com/timeless/video/the-worlds-columbian-exposition/3452716 -s -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['http://www.nbc.com/timeless/video/the-worlds-columbian-exposition/3452716', '-s', '-v']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2017.02.07
[debug] Python version 3.4.4 - Windows-10-10.0.14393
[debug] exe versions: ffmpeg 3.2.2, ffprobe 3.2.2, rtmpdump 2.3
[debug] Proxy map: {}
[NBC] 3452716: Downloading webpage
[ThePlatform] 3452716: Downloading webpage
[ThePlatform] 3452716: Downloading SMIL data
ERROR: This content requires a valid, unexpired auth token.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpfjt892j_\build\youtube_dl\YoutubeDL.py", line 696, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpfjt892j_\build\youtube_dl\extractor\common.py", line 369, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpfjt892j_\build\youtube_dl\extractor\theplatform.py", line 274, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpfjt892j_\build\youtube_dl\extractor\theplatform.py", line 42, in _extract_theplatform_smil
youtube_dl.utils.ExtractorError: This content requires a valid, unexpired auth token.
<end of log>

Description of your issue, suggested solution and other information

Some NBC videos require Adobe authorization, which is currently not supported.

@gkoelln
Copy link
Contributor Author

@gkoelln gkoelln commented Feb 7, 2017

Yes, I'm aware of the option, thank you, but when I add my locally updated MSO:

[NBC] 3142497: Downloading webpage
[ThePlatform] 3142497: Downloading webpage
[ThePlatform] 3142497: Downloading SMIL data
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--ap-mso', 'Dish', '--ap-username', 'PRIVATE', '--ap-password', 'PRIVATE', 'http://www.nbc.com/timeless/video/party-at-castle-varlar/3142497', '-s', '-v']
[debug] Encodings: locale cp1252, fs utf-8, out cp1252, pref cp1252
[debug] youtube-dl version 2017.02.01
[debug] Python version 3.6.0 - Windows-10-10.0.14393-SP0
[debug] exe versions: ffmpeg 3.2.2, ffprobe 3.2.2, rtmpdump 2.3
[debug] Proxy map: {}
ERROR: This content requires a valid, unexpired auth token.
Traceback (most recent call last):
  File "C:\python36\lib\youtube-dl\youtube_dl\YoutubeDL.py", line 694, in extract_info
    ie_result = ie.extract(url)
  File "C:\python36\lib\youtube-dl\youtube_dl\extractor\common.py", line 369, in extract
    return self._real_extract(url)
  File "C:\python36\lib\youtube-dl\youtube_dl\extractor\theplatform.py", line 274, in _real_extract
    formats, subtitles = self._extract_theplatform_smil(smil_url, video_id)
  File "C:\python36\lib\youtube-dl\youtube_dl\extractor\theplatform.py", line 42, in _extract_theplatform_smil
    raise ExtractorError(error_element.attrib['abstract'], expected=True)
youtube_dl.utils.ExtractorError: This content requires a valid, unexpired auth token.

So, it's not even trying to authenticate, or get an auth token.

@gkoelln
Copy link
Contributor Author

@gkoelln gkoelln commented Feb 7, 2017

So, IMO, we need to add the authentication to either NBC or theplatform. It's just a bit above me, or I'd do it myself.

@yan12125
Copy link
Collaborator

@yan12125 yan12125 commented Feb 7, 2017

Oops, I forgot you're the author of Dish authenticator. Yes nbc.py needs an update.

@yan12125 yan12125 added the request label Feb 7, 2017
@remitamine remitamine self-assigned this Feb 8, 2017
@remitamine remitamine closed this in fdf9b95 Feb 8, 2017
@gkoelln
Copy link
Contributor Author

@gkoelln gkoelln commented Feb 8, 2017

Thanks!

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
3 participants
You can’t perform that action at this time.