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

ArteTvIE is broken #875

Closed
jaimeMF opened this issue Jun 6, 2013 · 4 comments
Closed

ArteTvIE is broken #875

jaimeMF opened this issue Jun 6, 2013 · 4 comments
Labels

Comments

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Jun 6, 2013

Reported by @screen12345, he suggested to change the _VALID_URL to (?:http://)?www\.arte.tv/guide/(?:fr|de)/[0-9-](?:/.*)?

@screen12345
Copy link

@screen12345 screen12345 commented Jun 26, 2013

l'm afraid that it doesn't work... Just tried it with

Just discovered that there are different URL schemes...
URL of single programme:
http://www.arte.tv/guide/de/050047-000/preiskampf-was-ist-unser-essen-wert?autoplay=1
(links expire after 7 days. here's more: http://www.arte.tv/guide/de/plus7)

But the daily evening news has the following URL:
http://www.arte.tv/guide/de/sendungen/AJT/arte-journal?autoplay=1 (German)
http://www.arte.tv/guide/fr/emissions/AJT/arte-journal?autoplay=1 (French)
Those would also have to be supported...

Finally, there are some variable parts in the URL for the corresponding audience such as de and fr as well as sendungen and emissions.

so maybe something like this:
(?:http://)?www\.arte.tv/guide/(?:fr|de)/(?:[0-9-](?:/.*)?|(?:(?:sendungen|emissions)/[a-zA-Z]+/.*)
That should cover both types of URLs...

@jaimeMF jaimeMF closed this in 75c9481 Jun 26, 2013
@jaimeMF
Copy link
Collaborator Author

@jaimeMF jaimeMF commented Jun 26, 2013

Thanks for the examples and the regex, I have fixed the info extractor.

@screen12345
Copy link

@screen12345 screen12345 commented Jun 27, 2013

Thanks! I hadn't realised that you had already fixed it.

@jaimeMF
Copy link
Collaborator Author

@jaimeMF jaimeMF commented Jun 27, 2013

Sorry for the confusion, but when you post the examples I hadn't fixed it, I have fixed it using the examples you post.

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.