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

http: Function to get Cookie value from request. (IDFGH-5548) #7273

Closed

Conversation

patrykkrz
Copy link
Contributor

Sometimes need get Cookie from http request.
I added function to get cookie value.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jul 14, 2021
@github-actions github-actions bot changed the title http: Function to get Cookie value from request. http: Function to get Cookie value from request. (IDFGH-5548) Jul 14, 2021
@Alvin1Zhang
Copy link
Collaborator

Thanks for your contribution.

@espressif-bot espressif-bot added Status: In Progress Work is in progress and removed Status: Opened Issue is new labels Jul 19, 2021
* - ESP_ERR_INVALID_ARG : Null arguments
* - ESP_ERR_HTTPD_RESULT_TRUNC : Value string truncated
*/
esp_err_t httpd_cookie_key_value(const char *cookie_str, const char *key, char *val, size_t val_size);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@patrykkrz Any reason for adding this API in header file?
Will this be used from application? Or it is just a helper API for httpd_req_get_cookie_val?

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: In Progress Work is in progress labels Jul 30, 2021
@Coffeeye Coffeeye deleted the master-get-cookie-from-req branch March 14, 2022 09:28
loganfin pushed a commit to Lumenaries/esp_http_server that referenced this pull request Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants