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

Add optional cookies argument #13

Closed
wants to merge 1 commit into from
Closed

Conversation

marblar
Copy link

@marblar marblar commented Mar 2, 2012

No description provided.

@jkbrzt
Copy link
Member

jkbrzt commented Mar 3, 2012

I am not sure if --headers is actually needed because it's just a header like any other.

When you want to send, for example, the following cookies:

Cookie: a=b;  c=d

You can do:

http GET example.org 'Cookie:a=b; c=d'

Which I would say is more intuitive than this:

http --cookies a=b --cookies c=d GET example.org

@jkbrzt jkbrzt closed this Mar 3, 2012
@marblar
Copy link
Author

marblar commented Mar 4, 2012

@jkbr Good point. Would it be worth adding the ability to automatically store/read cookies from a file? It might be convenient for lengthier browsing sessions.

@jkbrzt jkbrzt mentioned this pull request Mar 6, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or enhancement wontfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants