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

Why does setCookie require a URL when a Cookie has its domain and path set? #180

Open
issuefiler opened this issue Jan 1, 2020 · 5 comments
Assignees
Labels
FeatureRequest minor We expect this work to be a minor semver change
Projects

Comments

@issuefiler
Copy link

What’s the point? Cookies are filtered out when you do getCookies(url); so really, what is the point?

@awaterma
Copy link
Member

awaterma commented Jan 6, 2020

I think this would make a good enhancement; we do parse the cookie once it's been passed. I'll bring this up for discussion.

@awaterma awaterma added FeatureRequest minor We expect this work to be a minor semver change labels Jan 6, 2020
@dennismphil dennismphil self-assigned this Jan 13, 2020
@awaterma awaterma added this to To do in 4.2 Release via automation Mar 21, 2022
@GrinZero
Copy link

I think this is stupid. There are all kinds of data in the cookie. Why do I need to pass other parameters? Do setcookie need any additional parameters?

@awaterma
Copy link
Member

Let's try and keep the language in our repository positive, @GrinZero! We explain these standards in our code of conduct here:

https://github.com/salesforce/tough-cookie/blob/30246e6c039f91ca33fab2046ffdf7a2d3d8c33c/CODE_OF_CONDUCT.md#our-standards

Please let me know if you have a pull request or change you would like to submit; we are slowly looking at and working over these issues as time goes on, but would love more help from the community.

@GrinZero
Copy link

GrinZero commented Apr 1, 2022

Sorry, I just want to highlight that this is unreasonable, not to vent my emotions

@awaterma
Copy link
Member

awaterma commented Apr 1, 2022

I'll try and pick up this issue when I can next @GrinZero; I think @dennismphil got busy with other work, and we left this behind. But I agree; our API should be easier to use, and not require redundant information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FeatureRequest minor We expect this work to be a minor semver change
Projects
No open projects
Development

No branches or pull requests

4 participants