hapijs/cookie

Update to README

grahamnordstrom opened this issue · 3 comments

grahamnordstrom commented

I have figured out the issue. When I declared the parameter, never before specified the path. Now with version 10 the path will be defaulted for the specific url path, that is why I could not see the cookie on the new url, because only applied to the previous old one. I had to add path: '/' to the initial options and it started to work.

Originally posted by @CloudVector in #211 (comment)

grahamnordstrom commented

Would be helpful to mention this in the README or in the upgrade path.
Alternatively the path could default to '/' as it appears that the cookie isn't set / accessible without it being explicitly set.

  • 👍1
hueniverse commented

This was noted in the release notes but lacked full explanation of the impact. I updated the notes.

lock commented

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.