req and res should be bound inside the new context

[coltrane]

The existing code binds req and res before starting a new context:

module.exports = function clsify(ns) {
  if (!ns) throw new Error('CLS namespace required');

  return function (req, res, next) {
    ns.bindEmitter(req);
    ns.bindEmitter(res);

    ns.run(function () {
      next();
    });
  };
};

But it seems that these should be bound within the new context. Something like this...

module.exports = function clsify(ns) {
  if (!ns) throw new Error('CLS namespace required');

  return function (req, res, next) {
    ns.run(function () {
      ns.bindEmitter(req);
      ns.bindEmitter(res);
      next();
    });
  };
};

[marcelboettcher]

+1