flags icon indicating copy to clipboard operation
flags copied to clipboard

Enable concise syntax of `response.cookie(...flagBag.cookie)`

Open dferber90 opened this issue 3 years ago • 1 comments

This originally existed on an unreleased branch in 77bfb6ff3f22167c8655987ff44aaa975bf5f292 but had to be removed as a TypeScript warning appeared on the calling side of response.cookie(...flagBag.cookie):

A spread argument must either have a tuple type or be passed to a rest parameter.

The code works, but I could not figure out the correct type.

Reintroducing this would allow developers to use response.cookie(...flagBag.cookie) instead of response.cookie(...flagBag.cookie.args).

dferber90 avatar Nov 22 '21 07:11 dferber90

flagBag.setCookie(response) would be a nicer way of handling this. Then we can drop flagBag.cookie.args.

dferber90 avatar Jan 15 '22 09:01 dferber90