Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Basic Usage Example Missing "next()" #135

Open
terrymorse58 opened this issue Aug 27, 2018 · 0 comments
Open

Basic Usage Example Missing "next()" #135

terrymorse58 opened this issue Aug 27, 2018 · 0 comments

Comments

@terrymorse58
Copy link

terrymorse58 commented Aug 27, 2018

The "Basic Usage" example code is missing a next() function call, breaking the middleware chain:

app.use(function(req, res, next) {
if (req.mySession.seenyou) {
res.setHeader('X-Seen-You', 'true');
} else {
// setting a property will automatically cause a Set-Cookie response
// to be sent
req.mySession.seenyou = true;
res.setHeader('X-Seen-You', 'false');
}
});

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant