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

Feature/extra - Let the user login & fetch an episode in 720p #61

Closed
wants to merge 3 commits into from

Conversation

bvanheu
Copy link
Owner

@bvanheu bvanheu commented Jun 9, 2015

Sup guys,

I've implemented a basic auth module to let people fetch the HD content from the extra section.

I didn't find a way to fetch the extra content, but at least one can download any supported episode in 720p so I thought we could at least push this into the repository.

I have extended the CLI with the 'login' command. You can try it by running:

$ toutv login
Password:

Then:

$ toutv -n info 2424966441 2425153505

Review & comment please!

@bvanheu bvanheu changed the title Feature/extra - Let the user login & fetch an epsiode in 720p Feature/extra - Let the user login & fetch an episode in 720p Jun 9, 2015
r = requests.get(url, params=params, headers=toutv.config.HEADERS,
headers = dict(toutv.config.HEADERS)

if auth and params:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why "auth and param"?

What happens if no params are passed, but we still want to authenticate?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only case where we don't have any params is when we download the segments from the Akamai server. I just wanted to avoid an uncessary HTTP request to get a claims.

@gboudreau
Copy link
Contributor

Quick note: login command wasn't working for me. I fixed it, and made a PR with my fix (in addition to a rebase of the feature/extra branch on the latest master).

@gboudreau
Copy link
Contributor

This PR is now deprecated; this functionality was merged into master a while ago. See #79

@gboudreau gboudreau closed this Jan 12, 2017
@gboudreau gboudreau deleted the feature/extra branch January 12, 2017 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants