top | item 12455365

(no title)

abimaelmartell | 9 years ago

This feels so wrong....

    cookies({ token: '42' }, {
      expires: 100 * 24 * 3600,     // The time to expire in seconds
      domain: false,                // The domain for the cookie
      path: '/',                    // The path for the cookie
      secure: https ? true : false  // Require the use of https
    });

discuss

order

SamBam|9 years ago

Why? It looks pretty standard to me. It uses named variables for options, instead of argument position, and has reasonable defaults, so you don't have to specify every option.

This is the standard, modern way of added options to a function.

niftich|9 years ago

Can you elaborate? Is it the DSL-like syntax for setting cookies that's strange? Or?

I actually quite like it because you can manipulate each portion of the string separetely, vs.

  token: 42; expires: 8640000; path=/; secure