Something better than req.accepts('json') && !req.accepts('html') ?? about accepts HOT 10 CLOSED

You probably want this:

switch (req.accepts('html', 'json')) {
  case 'json':
    // process the GET request from superagent (client)
    // ..
    res.json(200, {});
    break;
  default:
    // middleware further down will load the basic page layout
    next();
    break;
}

from accepts.

yeah i dunno why you would ever want to do that... do a switch

from accepts.

...and the bonus of using the switch method above means you are interpreting the Accept header correctly, unlike the original if statement (the if statement will execute the else statement for Accept: application/json, */*;q=0.1, for example, which is probably not what you wanted to happen).

from accepts.

Hmmm... @dougwilson I do not know. I am not a fan of switch blocks because they are not OO friendly and sometimes complicate things.

How about something like req.acceptsOnly('json')? That would be neat!

from accepts.

that's not how content negotiation works though

from accepts.

@binarykitchen a req.acceptsOnly('json') would still return false for Accept: application/json, */*;q=0.1, which is unlikely what you want. Also, there is nothing "not OO friendly" about switch statements. They are basically fancier if statements. You can always write it as an if if you like...

if (req.accepts('html', 'json') === 'json') {
  // process the GET request from superagent (client)
  // ..
  res.json(200, {});
} else {
  // middleware further down will load the basic page layout
  next();
}

but that if pattern will not scale very well if you want to add more.

from accepts.

Ah, I see now ... thanks so much!

Can I somehow plug in a new method to the request instance through the whole express app which looks like this:

req.wantsJson = function() {
    return this.accepts('html', 'json') === 'json')
}

Where could I do that? I am keen to avoid code duplication in all my controllers hence the need for such a common function.

from accepts.

function wantsJson() {
  return this.accepts('html', 'json') === 'json';
}
app.use(function (req, res, next) {
  req.wantsJson = wantsJson;
  next();
});

is the best way to do it. Note that having the wantsJson outside the middleware is how it will perform well since it doesn't create a new function for every request.

from accepts.

Thanks so much @dougwilson - after all your help I owe you one beer!

from accepts.

🍺

from accepts.