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

document cookie format in CURLINFO_COOKIELIST #4805

Closed
bsammon opened this issue Jan 10, 2020 · 2 comments
Closed

document cookie format in CURLINFO_COOKIELIST #4805

bsammon opened this issue Jan 10, 2020 · 2 comments

Comments

@bsammon
Copy link

bsammon commented Jan 10, 2020

It would be nice if the CURLINFO_COOKIELIST had some documentation as to the format of the cookie strings that it returns, or a pointer to such documentation. (is it a standard format?)

By experimentation, I've determined that it seems to be tab-delimited, with a predictable ordering of the fields.

@bagder
Copy link
Member

bagder commented Jan 10, 2020

I agree, it should be mentioned there or point to where the documentation for it exists.

Currently, it is at least documented in Everything curl: https://ec.haxx.se/libcurl-http/libcurl-http-cookies#cookie-file-format

@jay
Copy link
Member

jay commented Jan 10, 2020

Also see the example in https://curl.haxx.se/libcurl/c/CURLOPT_COOKIELIST.html

bagder added a commit that referenced this issue Jan 10, 2020
... and refer to that file from from CURLOPT_COOKIEFILE.3 and
CURLOPT_COOKIELIST.3

Reported-by: bsammon on github
Fixes #4805
@bagder bagder self-assigned this Jan 11, 2020
bagder added a commit that referenced this issue Jan 13, 2020
... and refer to that file from from CURLOPT_COOKIEFILE.3 and
CURLOPT_COOKIELIST.3

Reported-by: bsammon on github
Fixes #4805
@bagder bagder closed this as completed in 9d1a092 Jan 13, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Apr 15, 2020
@bagder bagder added the cookies label Nov 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging a pull request may close this issue.

3 participants