What's the purpose of the options.sameSite branch of ternary in serialize?

[jonchurch]

What's the purpose of the options.sameSite branch of ternary in serialize?

When options.priority is not a string, it will use options.sameSite. There's no overlap in the types between the two, so we'd hit the default and throw. Was there meant to be overlap or is this a typo?

cookie/src/index.ts

Line 314 in 408c2b4

: options.sameSite;

  if (options.priority) {
    const priority =
      typeof options.priority === "string"
        ? options.priority.toLowerCase()
        : options.sameSite; // typo?
    switch (priority) {
      case "low":
        str += "; Priority=Low";
        break;
      case "medium":
        str += "; Priority=Medium";
        break;
      case "high":
        str += "; Priority=High";
        break;
      default:
        throw new TypeError(`option priority is invalid: ${options.priority}`);
    }
  }

[jonchurch]

It's functionally sound, non strings will throw with the correct error, except in the edgecase where someone did {priority: 1, sameSite: "low"}

That still throws, but only once we validate options.sameSite