-
Notifications
You must be signed in to change notification settings - Fork 16
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
There is no ability to add additional params to cookies #48
Comments
You can use a custom
|
At this moment you cannot set a custom path for cookies. Do you wish to open a PR for it? Contributions welcome. |
I honestly feel like the current ability to pass in your own |
Okey, but how do we specify if we want a secure cookie? Or when should it expire? |
I wish I could set expiry in a simpler way, like this:
I really don't want to implement my own setCookie since all I wanted is to change only the expire time and it'd be cool to be able to set different expiry times for different cookies. May I implement this or this library is not suppose to work this way? |
@lucasjmatias felt exactly the same. Especially that changing expiry shouldnt warrant providing as new setCookies implementation. Did you do anything regarding this? |
How to add
expires
orpath
param when we set cookies?The text was updated successfully, but these errors were encountered: