misc cookies

[timotheecour]

links

• jester: https://github.com/dom96/jester/blob/master/jester/private/utils.nim
• prologue: https://github.com/planety/prologue/blob/f61acc5bbab2d304c8b36fb25cd959714ca712d1/src/prologue/core/response.nim#L63
• https://github.com/planety/cookiejar/blob/master/src/cookiejar/cookies.nim
• std/cookies cookies
• [stdlib] update cookies module by xflywind · Pull Request #17116 · nim-lang/Nim

[timotheecour]

• set cookie expiration date

TODO: max-age
http://promincproductions.com/blog/set-cookie-expiration-date-browser-compatiability/
using maybe:

proc formatExpires*(time: Time): string =
  result = format(time.utc, "ddd',' dd MMM yyyy HH:mm:ss 'GMT'")

[timotheecour]

• addCookie to set multiple cookies in a single web request

IIRC I needed something addCookie when using jester, and jester's setCookie wasn't flexible enough, probably because of Unable to set multiple cookies in a single web request · Issue #224 · dom96/jester

template setCookie*(name, value: string, expires="",
                    sameSite: SameSite=Lax, secure = false,
                    httpOnly = false, domain = "", path = "") =

here's what I wrote instead:

proc addCookie*(headers: var Option[RawHeaders], cookie: string) =
  # note: requires makeCookie eg: (key, value, expires: string, domain = "", path = "", secure = false, httpOnly = false, sameSite = Lax): string =
  # EG: addCookie(result[2], cookie)
  if isSome(headers) and
     (let headers2 = headers.get(); headers2.toTable.hasKey("Set-Cookie")):
    headers = some(headers2 & @({"Set-Cookie": cookie}))
  else:
    # Note: requires modif to jester
    setHeader(headers, "Set-Cookie", cookie)

[timotheecour]

• removeCookie

I use something like this (adapted from jester API/code)

proc removeCookie*(headers: var Option[RawHeaders], cookieKey: string, path = "") =
  ## https://stackoverflow.com/questions/5285940/correct-way-to-delete-cookies-server-side
  let expires = "Thu, 01 Jan 1970 00:00:00 GMT"
  # consider using fixed key for cookie value
  let cookie = makeCookie(cookieKey, "", expires = expires, path = path)
  addCookie(headers = headers, cookie = cookie)

[timotheecour]

• using StringTableRef doesn't seem correct for parseCookies because IIUC we can use duplicate keys (eg with different paths etc):

nim r --eval:'import std/[cookies,strtabs]; echo parseCookies("a=1;Path=/; a=2;Path=/example;")' prints {a: 1, Path: /} which seems wrong according to https://stackoverflow.com/questions/4056306/how-to-handle-multiple-cookies-with-the-same-name

[timotheecour]

• https://github.com/nim-lang/Nim/pull/17116/files#r583093794

[timotheecour]

• SameSite naming https://github.com/nim-lang/Nim/pull/17116/files#r583085830