Skip to content

/ youtube-dl

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

Jump to bottom

InfoQ extractor too restrictive #6071

Closed
Dridi opened this issue Jun 24, 2015 · 3 comments
Closed

InfoQ extractor too restrictive #6071

Dridi opened this issue Jun 24, 2015 · 3 comments

Comments

@Dridi
Copy link

@Dridi Dridi commented Jun 24, 2015

Hi,

InfoQ contents can be localized, and it would be great and not too hard to take it into account.

Lazy fix, without checking supported languages:

_VALID_URL = r'https?://(?:www\.)?infoq\.com/(\w\w/)?[^/]+/(?P<id>[^/]+)$'

Cheers
@dstftw
Copy link
Collaborator

@dstftw dstftw commented Jun 24, 2015

Post an example URL it does not catch.
@Dridi
Copy link
Author

@Dridi Dridi commented Jun 25, 2015

Today's first presentation on the French front page:
http://www.infoq.com/fr/presentations/changez-avis-sur-javascript
@dstftw dstftw closed this in 533f67d Jun 25, 2015
@Dridi
Copy link
Author

@Dridi Dridi commented Jun 25, 2015

Thanks!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

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