Skip to content

Improve redirect handling#157

Merged
fafhrd91 merged 2 commits intoaio-libs:masterfrom
Eyepea:improve_redirect_handling
Oct 9, 2014
Merged

Improve redirect handling#157
fafhrd91 merged 2 commits intoaio-libs:masterfrom
Eyepea:improve_redirect_handling

Conversation

@ludovic-gasc
Copy link
Copy Markdown
Contributor

Hi,

I've added 307 handling (#155) + for 301 and 302, I mimic IE behaviour, to replace HTTP verb by GET after redirection. For technical details: https://github.com/kennethreitz/requests/pull/269

It's a really an issue for us, because we need to interact with some old fashion webapp without WS API that use a lot this assumption.
Moreover, this behaviour is now official, 307 HTTP code has been created to handle correctly redirections.

Thanks for reviewing.

Regards.

fafhrd91 added a commit that referenced this pull request Oct 9, 2014
@fafhrd91 fafhrd91 merged commit e494364 into aio-libs:master Oct 9, 2014
@fafhrd91
Copy link
Copy Markdown
Member

fafhrd91 commented Oct 9, 2014

Thank you!

@ludovic-gasc
Copy link
Copy Markdown
Contributor Author

Hi @fafhrd91,

Thanks for the merge, it's a real pleasure to work with a collaborative guy ;-)

BTW, if somebody could make a release of aiohttp in a near future, it should be cool.
Not a stress for us, we use the patched version in github via requirements.txt, but for newbies on aiohttp, they will have quickly this issue if they use aiohttp for web scrapping.

@ludovic-gasc ludovic-gasc deleted the improve_redirect_handling branch October 9, 2014 12:24
@asvetlov
Copy link
Copy Markdown
Member

asvetlov commented Oct 9, 2014

+1 for next bugfix release

On Thu, Oct 9, 2014 at 3:24 PM, Ludovic Gasc [email protected]
wrote:

Hi @fafhrd91 https://github.com/fafhrd91,

Thanks for the merge, it's a real pleasure to work with a collaborative
guy ;-)

BTW, if somebody could make a release of aiohttp in a near future, it
should be cool.
Not a stress for us, we use the patched version in github via
requirements.txt, but for newbies on aiohttp, they will have quickly this
issue if they use aiohttp for web scrapping.


Reply to this email directly or view it on GitHub
#157 (comment).

Thanks,
Andrew Svetlov

@fafhrd91
Copy link
Copy Markdown
Member

fafhrd91 commented Oct 9, 2014

I'd like to merge PR for chunking requests first. I am planing to make
release later next week.

On Thursday, October 9, 2014, Andrew Svetlov [email protected]
wrote:

+1 for next bugfix release

On Thu, Oct 9, 2014 at 3:24 PM, Ludovic Gasc <[email protected]
javascript:_e(%7B%7D,'cvml','[email protected]');>
wrote:

Hi @fafhrd91 https://github.com/fafhrd91,

Thanks for the merge, it's a real pleasure to work with a collaborative
guy ;-)

BTW, if somebody could make a release of aiohttp in a near future, it
should be cool.
Not a stress for us, we use the patched version in github via
requirements.txt, but for newbies on aiohttp, they will have quickly
this
issue if they use aiohttp for web scrapping.


Reply to this email directly or view it on GitHub
#157 (comment).

Thanks,
Andrew Svetlov


Reply to this email directly or view it on GitHub
#157 (comment).

@lock
Copy link
Copy Markdown

lock bot commented Oct 30, 2019

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.

@lock lock bot added the outdated label Oct 30, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants