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

redirect with cookies and headers #19

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

redirect with cookies and headers #19

poweroftrue opened this issue Nov 25, 2015 · 10 comments
Assignees
Labels

Comments

@poweroftrue
Copy link

like other http libs for golang
when redirect cookies disappear

@jeevatkm jeevatkm self-assigned this Nov 26, 2015
@jeevatkm
Copy link
Member

@mostafadahab - You may have to set HTTP mode to take advantage for cookiejar capability. Please have a look here. Default mode is REST.

@poweroftrue
Copy link
Author

Not only the cookies the headers also changed:
image
And when the resty follows request:
image

@jeevatkm
Copy link
Member

Thanks for inputs, will have a look and let you know.

@poweroftrue
Copy link
Author

franela/goreq#69

I think this will help

@jeevatkm
Copy link
Member

@mostafadahab Thank you for your patient. I was bit occupied in the day work.

Currently I'm working on this issue and other request from your end. I will update you!

@jeevatkm jeevatkm added bug and removed analysis labels Dec 27, 2015
@jeevatkm jeevatkm added this to the v0.5 Milestone milestone Dec 27, 2015
@jeevatkm jeevatkm changed the title redirect with cookies redirect with cookies and headers Dec 27, 2015
@jeevatkm
Copy link
Member

I went through given reference and golang/go#4800 for redirect.

go-resty will add all the headers during a redirect for the same host. It is available in master. Can you please try and share you feedback on this issue?

@jeevatkm
Copy link
Member

Did you get a chance to have a look? Thanks.

@poweroftrue
Copy link
Author

I'll check this weekend

@jeevatkm
Copy link
Member

Okay, sounds good.

@jeevatkm jeevatkm mentioned this issue Jan 2, 2016
@jeevatkm
Copy link
Member

jeevatkm commented Jan 4, 2016

done!

@jeevatkm jeevatkm closed this as completed Jan 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

2 participants