Correct to set cookie.httpOnly option to false

[hannut91]

Even if the httpOnly option is set to false, it is set to true.
The logical OR (||) operator (logical disjunction) for a set of operands is true if and only if one or more of its operands is true.
So changed to use Nullish coalescing operator(??).

Issue: #353

See also

• https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Logical_OR
• https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Nullish_coalescing_operator

[changeset-bot]

⚠️ No Changeset found

Latest commit: 3e6bfb4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[codecov]

Codecov Report

Merging #354 (3e6bfb4) into master (a096734) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #354   +/-   ##
=======================================
  Coverage   99.17%   99.17%           
=======================================
  Files           6        6           
  Lines         121      121           
  Branches       37       37           
=======================================
  Hits          120      120           
  Misses          1        1           

Impacted Files	Coverage Δ
src/session.ts	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a096734...3e6bfb4. Read the comment docs.

[hoangvvo]

I noticed that I have used it elsewhere anyway, so the previous comment is invalid!

[hoangvvo]

LGTM! Thanks!

[hoangvvo]

Published as v4.0.3. Thanks again for the PR and the very detailed description!