Skip to content

docs: use const instead of var in README examples #1143

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Hashim1999164 wants to merge 1 commit into
base: master
Choose a base branch
from
Open

docs: use const instead of var in README examples #1143

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 16 additions & 16 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ $ npm install express-session
## API

```js
var session = require('express-session')
const session = require('express-session')
```

### session(options)
Expand Down Expand Up @@ -52,13 +52,13 @@ Settings object for the session ID cookie. The default value is
In addition to providing a static object, you can also pass a callback function to dynamically generate the cookie options for each request. The callback receives the `req` object as its argument and should return an object containing the cookie settings.

```js
var app = express()
const app = express()
app.use(session({
secret: 'keyboard cat',
resave: false,
saveUninitialized: true,
cookie: function(req) {
var match = req.url.match(/^\/([^/]+)/);
const match = req.url.match(/^\/([^/]+)/);
return {
path: match ? '/' + match[1] : '/',
httpOnly: true,
Expand Down Expand Up @@ -193,7 +193,7 @@ have your node.js behind a proxy and are using `secure: true`, you need to set
"trust proxy" in express:

```js
var app = express()
const app = express()
app.set('trust proxy', 1) // trust first proxy
app.use(session({
secret: 'keyboard cat',
Expand All @@ -207,8 +207,8 @@ For using secure cookies in production, but allowing for testing in development,
the following is an example of enabling this setup based on `NODE_ENV` in express:

```js
var app = express()
var sess = {
const app = express()
const sess = {
secret: 'keyboard cat',
cookie: {}
}
Expand Down Expand Up @@ -480,7 +480,7 @@ to adjust the `.expires` property appropriately. The following
are essentially equivalent

```js
var hour = 3600000
const hour = 3600000
req.session.cookie.expires = new Date(Date.now() + hour)
req.session.cookie.maxAge = hour
```
Expand Down Expand Up @@ -906,11 +906,11 @@ based session store. Supports all backends supported by Fortune (MongoDB, Redis,
A simple example using `express-session` to store page views for a user.

```js
var express = require('express')
var parseurl = require('parseurl')
var session = require('express-session')
const express = require('express')
const parseurl = require('parseurl')
const session = require('express-session')

var app = express()
const app = express()

app.use(session({
secret: 'keyboard cat',
Expand All @@ -924,7 +924,7 @@ app.use(function (req, res, next) {
}

// get the url pathname
var pathname = parseurl(req).pathname
const pathname = parseurl(req).pathname

// count the views
req.session.views[pathname] = (req.session.views[pathname] || 0) + 1
Expand All @@ -948,11 +948,11 @@ app.listen(3000)
A simple example using `express-session` to keep a user log in session.

```js
var escapeHtml = require('escape-html')
var express = require('express')
var session = require('express-session')
const escapeHtml = require('escape-html')
const express = require('express')
const session = require('express-session')

var app = express()
const app = express()

app.use(session({
secret: 'keyboard cat',
Expand Down