Skip to content
This repository has been archived by the owner on Jan 20, 2021. It is now read-only.

Unable to use offsets if it's formatted as &t vs ?t in the URL. #111

Closed
fiveofeight opened this issue Jan 14, 2016 · 2 comments
Closed

Unable to use offsets if it's formatted as &t vs ?t in the URL. #111

fiveofeight opened this issue Jan 14, 2016 · 2 comments
Labels

Comments

@fiveofeight
Copy link
Contributor

Currently if you try to use an offset such as: https://www.youtube.com/watch?v=8GznlD5jTqY&t=1m04s it does not work, it only works when formatted as https://youtu.be/8GznlD5jTqY?t=1m50s

https://github.com/matthieugrieger/mumbledj/blob/master/service_youtube.go#L91
Changing from:
offset: int(yt.parseTime(offset,?T=(?P\d+D)?(?P\d+H)?(?P\d+M)?(?P\d+S)?).Seconds()),
to
offset: int(yt.parseTime(offset,T=(?P\d+D)?(?P\d+H)?(?P\d+M)?(?P\d+S)?).Seconds()),
resolves the issue.

@matthieugrieger
Copy link
Owner

Ah, good catch. Would you mind making a pull request for this?

@fiveofeight
Copy link
Contributor Author

Done.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants