Blip.tv #386

Merged
merged 6 commits into from Aug 1, 2012

Conversation

Projects
None yet
4 participants
@FiloSottile
Collaborator

FiloSottile commented Aug 1, 2012

This includes the pull requests #356, #357 and #358 by @jcarlosgarciasegovia plus a fix to make the User-Agent change specific to only the BlipTV IE and a BlipTVUserIE fix/refresh. Fixes #325.

phihag added a commit that referenced this pull request Aug 1, 2012

@phihag phihag merged commit a171dbf into rg3:master Aug 1, 2012

@rbrito

This comment has been minimized.

Show comment Hide comment
@rbrito

rbrito Aug 5, 2012

Contributor

@phihag, @FiloSottile

For activist reasons, we should not pretend to always be iTunes.

For technical reasons, we should allow information extractors to override the default user-agent from the std_headers dictionary, as it is not impossible to have one site (like blip.tv) serve the content for user-agent A, with another site serving content for user-agent B, with A != B, of course. :)

Contributor

rbrito commented on youtube_dl/utils.py in 3c4fc58 Aug 5, 2012

@phihag, @FiloSottile

For activist reasons, we should not pretend to always be iTunes.

For technical reasons, we should allow information extractors to override the default user-agent from the std_headers dictionary, as it is not impossible to have one site (like blip.tv) serve the content for user-agent A, with another site serving content for user-agent B, with A != B, of course. :)

@rbrito

This comment has been minimized.

Show comment Hide comment
@rbrito

rbrito Aug 5, 2012

Contributor

Ah, great.

Ah, great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment