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

feat(cookie): Added the support for http cookie #473

Closed
wants to merge 1 commit into from

Conversation

userAdityaa
Copy link
Contributor

PR Description

This PR introduces support for sending HTTP cookies with API requests. The new functionality allows users to include cookies in their headers, which is especially useful when interacting with endpoints that rely on session tokens or other cookie-based authentication.

Related Issues

Checklist

  • I have gone through the contributing guide
  • I have updated my branch and synced it with project main branch before making this PR
  • I have run the tests (flutter test) and all tests are passing

Added/updated tests?

We encourage you to add relevant test cases.

  • Yes
  • No, and this is why: please replace this line with details on why tests have not been included

@ashitaprasad
Copy link
Member

The PR is incorrect as you have added Cookie support to a java Codegen, but we were looking for cookie support in the app. Also, cookies have to be retrieved from the reponse model headers of a request.

Closing it for now as this feature will require more discussion on the issue thread.

@Clasherzz
Copy link
Contributor

Clasherzz commented Nov 10, 2024

After extracting what exactly should be the feauture that needs to be implemented. Should it be stored according to the domain name like in postman . And upon clicking the cookies should they be added to the headers automatically for sending as request next time. Also can i maybe make a draft request and try to do this part by part while getting an overview about this idea rather than trying to implement the whole approach at one go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for HTTP Cookies
3 participants