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

Add support for www.rtlxl.nl #3523

Closed
pvdl opened this issue Aug 15, 2014 · 12 comments
Closed

Add support for www.rtlxl.nl #3523

pvdl opened this issue Aug 15, 2014 · 12 comments
Assignees

Comments

@pvdl
Copy link
Contributor

@pvdl pvdl commented Aug 15, 2014

Example:
http://www.rtlxl.nl/#!/rtl-nieuws-132237/6e4203a6-0a5e-3596-8424-c599a59e0677

rtl-nieuws-132237 : name of the program with a number
6e4203a6-0a5e-3596-8424-c599a59e0677 : I think it is a kind of identifier.

This site has also paid video's. You need Microsoft Silverlight.

Be aware that the '!' in the URL gives an error!
youtube-dl http://www.rtlxl.nl/#!/rtl-nieuws-132237/6e4203a6-0a5e-3596-8424-c599a59e0677
You can solve it by 'escaping' it to put a '' before the '!'.

@pvdl
Copy link
Contributor Author

@pvdl pvdl commented Aug 17, 2014

Little investigation:
rtl-nieuws-132237 : name of the program with a number
This number mentioned is not a unique number. It is a number which identifies a serie of programs.
6e4203a6-0a5e-3596-8424-c599a59e0677 is the unique ID of the program

@pvdl
Copy link
Contributor Author

@pvdl pvdl commented Aug 17, 2014

Regex for id: 'https?://www.rtlxl.nl/#!/[^/]+/(?P[^/?]+)'

@pvdl
Copy link
Contributor Author

@pvdl pvdl commented Aug 17, 2014

Example:

class RTLXL(InfoExtractor):
IE_NAME = 'rtlxl.nl'
_VALID_URL = r'https?://www.rtlxl.nl/#!/[^/]+/(?P[^/?]+)'
_TEST = {
'url': 'http://www.rtlxl.nl/#\!/hallo-holland-303393/157d5f20-f5e9-3b7d-a1ee-2583e232db69', # With URL escape
'md5': '???????', # To be determined.
'info_dict': {
'id': '157d5f20-f5e9-3b7d-a1ee-2583e232db69',
'title': 'Hallo Holland',
'description': 'Informatief programma waarin wordt getoond wat voor moois Nederland te bieden heeft. Daarnaast wordt er een gezin verrast met een weekendje weg en een paar leuke uitstapjes in eigen land.',
'upload_date': '20140817',
},
}
@pvdl
Copy link
Contributor Author

@pvdl pvdl commented Aug 17, 2014

@pvdl
Copy link
Contributor Author

@pvdl pvdl commented Aug 17, 2014

@pvdl
Copy link
Contributor Author

@pvdl pvdl commented Aug 17, 2014

@jaimeMF jaimeMF self-assigned this Aug 17, 2014
@pvdl
Copy link
Contributor Author

@pvdl pvdl commented Aug 17, 2014

Bas Rieter did awsome work for XBMC to get Dutch streams working.
Python code can be examined here: http://www.rieter.net/content/
/net.rieter.xot/deploy/net.rieter.xot.channel.rtlnl/rtl/

@jaimeMF jaimeMF closed this in 6493f5d Aug 18, 2014
@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Aug 18, 2014

Thanks for the report, it will be supported in the next version.

@phihag
Copy link
Contributor

@phihag phihag commented Aug 22, 2014

Thank you for the report. Support for rtlxl.nl has been added in youtube-dl 2014.08.22. Type youtube-dl -U to update.

@Reino17
Copy link

@Reino17 Reino17 commented Aug 24, 2014

jaimeMF @ #3145

It's actually possible to download the video in a single file, so that's what we use. But we also support the m3u8 protocol (which is used by default on npl.nl), so it would be easy to use them.

As a Dutchman I'm pleased to see support for npo.nl (public tv broadcast) and rtlxl.nl (commercial tv broadcast). I've noticed indeed, and am pleased to see, youtube-dl uses the progressive m4v singlefile videostream for npo.nl (which is for iOS and Android, or at least non-flash devices I believe).
For rtlxl.nl on the other hand you've only added support for the adaptive/dynamic videostreams. Why?

For http://www.rtlxl.nl/#!/rtl-nieuws-132237/6e4203a6-0a5e-3596-8424-c599a59e0677, the test-url in rtlnl.py, youtube-dl generates:
http://manifest.us.rtl.nl/rtlxl/v166/network/pc/flash/components/actueel/rtlnieuws/307455/2014w33/6e4203a6-0a5e-3596-8424-c599a59e0677.ssm/6e4203a6-0a5e-3596-8424-c599a59e0677.f4m
...but for non-flash devices rtlXL also has a progressive/static videostream:
http://pg.us.rtl.nl/rtlxl/network/a3m/progressive/components/actueel/rtlnieuws/307455/2014w33/6e4203a6-0a5e-3596-8424-c599a59e0677.ssm/6e4203a6-0a5e-3596-8424-c599a59e0677.mp4?f=ipad

This progressive/static url is generated by GemistDownloader, a piece of software (with a GUI) to download tv broadcasts. The author won't disclose how it generates this url. Sadly enough, I don't exactly know the inner workings of HTTP Live Streaming.
I'd much rather pipe the progressive mp4 to FFmpeg than the adaptive f4m. And besides, the f4m-file I can't open in Media Player Classic and VLC.
So I was wondering if you could have a look. I'd really appreciate it!

@phihag
Copy link
Contributor

@phihag phihag commented Aug 25, 2014

There is no global conspiracy to not add a single-file mp4 for rtlxl.nl, we simply didn't know that it would be there. We're happy to add more formats, but need a little more information.

Unfortunately, the rtlxl android app is not available from Germany.

Can you give us a packet dump of using the (android or iOS) app, or the apk file of the app? Alternatively, if you were willing to provide a VPN connection to one of the developers, we could for sure look whether we can add support for these ipad URLs.

Also note that this issue is already closed, and not identical to yours (the original poster just wanted any access, you want a specific way). In the future, please always open a new issue. That allows us to have an overview over which issues need tending to and which are done. It also allows us to relate commits to the issue, which enables a better understanding of the code afterwards. When you post in a closed issue, it's easy to overlook for us.

@Reino17
Copy link

@Reino17 Reino17 commented Aug 25, 2014

I never said there was. I was just wondering. :)

I don't know how to let it create such a packet dump, because I don't have much experience with Android, so HERE you can find version 3.2 of the RTL XL Android app. Seeing the changelog of the latest version 3.3 is just a minor change, I believe you can use 3.2 just as well.

You're right, next time I'll open a new issue.
Thank you.

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