Pass functions in as header values

[anthonator]

It would be nice if you could specify a function as a header value so you could have a set of default headers that may be calculated per request.

qwest.setDefaultOptions({
  headers: {
    'Accept': process.env.API_VERSION,
    'Authorization': () => `Bearer ${jwtFromLocalStorage()}`
  },
  responseType: 'json'
});

[pyrsmk]

If the Authorization header is updated at each request, then it shouldn't be a default option, isn't it? Maybe I don't understand well the problem ^^

[anthonator]

I'd argue it is a default option if every request requires it.

For my specific use case I need a header to be sent with a variable value always derived from the same function. However, that header needs to be sent on every request. From my perspective it's default behavior to always send that header.

[anthonator]

This is what I'm trying to avoid.

qwest.post('/things', {}, { headers: { 'MyHeader': myFunc() } })

qwest.get(`/things/${id}`, {}, { headers: { 'MyHeader': myFunc() } })

qwest.get('/things', {}, { headers: { 'MyHeader': myFunc() } })

... + 50 more

[pyrsmk]

Ok. I'll take a look at this as soon as possible ;)

[pyrsmk]

Or you can make a PR if you need this quickly

[pyrsmk]

It still seems weird to me to support that kind of feature in qwest since it just matches a really specific case.