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.

Sign up for GitHub

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

stop redirect #18

Closed
poweroftrue opened this issue Nov 25, 2015 · 3 comments
Closed

stop redirect #18

poweroftrue opened this issue Nov 25, 2015 · 3 comments
Assignees
@jeevatkm
Labels
bug
Milestone
v0.5 Milestone

Comments

@poweroftrue
Copy link

can resty stop redirect without getting error ?! i tried custom redirectpolicy but still nil continue the redirect and error panic and not returning with response
@jeevatkm jeevatkm self-assigned this Nov 27, 2015
jeevatkm added a commit that referenced this issue Dec 28, 2015
@jeevatkm
#18 - returns response too when redirect error
298f2a1
@jeevatkm jeevatkm added this to the v0.5 Milestone milestone Dec 28, 2015
@jeevatkm jeevatkm added bug and removed analysis labels Dec 28, 2015
@jeevatkm
Copy link
Member

@mostafadahab can you please try from master branch and share your feedback?

@poweroftrue
Copy link
Author

Good 👍 , worked for me and nice warnings

@jeevatkm
Copy link
Member

jeevatkm commented Jan 2, 2016

I'm glad to hear that. Thanks for verification. This change will be released with v0.5 release.

BTW, how about the headers while redirect #19 ?

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

@jeevatkm jeevatkm

Labels
bug
Milestone
v0.5 Milestone
Development

No branches or pull requests
2 participants
@jeevatkm @poweroftrue