Skip to content

Commit 7f88865

Browse files
authored
update for Fastify v5 (#154)
1 parent e943ad4 commit 7f88865

File tree

3 files changed

+86
-50
lines changed

3 files changed

+86
-50
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ on:
1717

1818
jobs:
1919
test:
20-
uses: fastify/workflows/.github/workflows/plugins-ci.yml@v3
20+
uses: fastify/workflows/.github/workflows/plugins-ci.yml@v4.1.0
2121
with:
2222
lint: true
2323
license-check: true

package.json

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -29,21 +29,21 @@
2929
},
3030
"homepage": "https://github.com/fastify/fastify-csrf#readme",
3131
"dependencies": {
32-
"@fastify/csrf": "^6.0.0",
33-
"@fastify/error": "^3.0.0",
34-
"fastify-plugin": "^4.0.0"
32+
"@fastify/csrf": "^6.3.0",
33+
"@fastify/error": "^3.4.1",
34+
"fastify-plugin": "^4.5.1"
3535
},
3636
"devDependencies": {
37-
"@fastify/cookie": "^9.0.4",
38-
"@fastify/pre-commit": "^2.0.2",
39-
"@fastify/secure-session": "^7.0.0",
40-
"@fastify/session": "^10.3.0",
41-
"@types/node": "^20.1.0",
42-
"fastify": "^4.2.0",
37+
"@fastify/cookie": "^9.3.1",
38+
"@fastify/pre-commit": "^2.1.0",
39+
"@fastify/secure-session": "^7.5.0",
40+
"@fastify/session": "^10.7.0",
41+
"@types/node": "^20.12.7",
42+
"fastify": "^4.26.2",
4343
"proxyquire": "^2.1.3",
44-
"sinon": "^17.0.0",
45-
"standard": "^17.0.0",
46-
"tap": "^16.0.0",
44+
"sinon": "^17.0.1",
45+
"standard": "^17.1.0",
46+
"tap": "^18.7.2",
4747
"tsd": "^0.31.0"
4848
},
4949
"pre-commit": [

test/user-info.test.js

Lines changed: 73 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -177,77 +177,113 @@ test('Validate presence of hmac key with User-Info /1', async (t) => {
177177
const fastify = Fastify()
178178
await fastify.register(fastifyCookie)
179179

180-
await t.rejects(fastify.register(fastifyCsrf, {
181-
getUserInfo (req) {
182-
return req.session.get('username')
183-
}
180+
await t.rejects(new Promise((resolve, reject) => {
181+
fastify.register(fastifyCsrf, {
182+
getUserInfo (req) {
183+
return req.session.get('username')
184+
}
185+
}).then(() => {
186+
resolve()
187+
}).catch(err => {
188+
reject(err)
189+
})
184190
}), Error('csrfOpts.hmacKey is required'))
185191
})
186192

187193
test('Validate presence of hmac key with User-Info /2', async (t) => {
188194
const fastify = Fastify()
189195
await fastify.register(fastifyCookie)
190196

191-
await t.rejects(fastify.register(fastifyCsrf, {
192-
getUserInfo (req) {
193-
return req.session.get('username')
194-
},
195-
sessionPlugin: '@fastify/cookie'
197+
await t.rejects(new Promise((resolve, reject) => {
198+
fastify.register(fastifyCsrf, {
199+
getUserInfo (req) {
200+
return req.session.get('username')
201+
},
202+
sessionPlugin: '@fastify/cookie'
203+
}).then(() => {
204+
resolve()
205+
}).catch(err => {
206+
reject(err)
207+
})
196208
}), Error('csrfOpts.hmacKey is required'))
197209
})
198210

199211
test('Validate presence of hmac key with User-Info /3', async (t) => {
200212
const fastify = Fastify()
201213
await fastify.register(fastifyCookie)
202214

203-
await t.rejects(fastify.register(fastifyCsrf, {
204-
getUserInfo (req) {
205-
return req.session.get('username')
206-
},
207-
csrfOpts: {
208-
hmacKey: undefined
209-
}
215+
await t.rejects(new Promise((resolve, reject) => {
216+
fastify.register(fastifyCsrf, {
217+
getUserInfo (req) {
218+
return req.session.get('username')
219+
},
220+
csrfOpts: {
221+
hmacKey: undefined
222+
}
223+
}).then(() => {
224+
resolve()
225+
}).catch(err => {
226+
reject(err)
227+
})
210228
}), Error('csrfOpts.hmacKey is required'))
211229
})
212230

213231
test('Validate presence of hmac key with User-Info /4', async (t) => {
214232
const fastify = Fastify()
215233
await fastify.register(fastifyCookie)
216234

217-
await t.rejects(fastify.register(fastifyCsrf, {
218-
getUserInfo (req) {
219-
return req.session.get('username')
220-
},
221-
sessionPlugin: '@fastify/cookie',
222-
csrfOpts: {
223-
hmacKey: undefined
224-
}
235+
await t.rejects(new Promise((resolve, reject) => {
236+
fastify.register(fastifyCsrf, {
237+
getUserInfo (req) {
238+
return req.session.get('username')
239+
},
240+
sessionPlugin: '@fastify/cookie',
241+
csrfOpts: {
242+
hmacKey: undefined
243+
}
244+
}).then(() => {
245+
resolve()
246+
}).catch(err => {
247+
reject(err)
248+
})
225249
}), Error('csrfOpts.hmacKey is required'))
226250
})
227251

228252
test('Validate presence of hmac key with User-Info /5', async (t) => {
229253
const fastify = Fastify()
230254
await fastify.register(fastifySecureSession, { key, cookie: { path: '/', secure: false } })
231255

232-
await t.resolves(fastify.register(fastifyCsrf, {
233-
getUserInfo (req) {
234-
return req.session.get('username')
235-
},
236-
sessionPlugin: '@fastify/secure-session'
256+
await t.resolves(new Promise((resolve, reject) => {
257+
fastify.register(fastifyCsrf, {
258+
getUserInfo (req) {
259+
return req.session.get('username')
260+
},
261+
sessionPlugin: '@fastify/secure-session'
262+
}).then(() => {
263+
resolve()
264+
}).catch(err => {
265+
reject(err)
266+
})
237267
}))
238268
})
239269

240270
test('Validate presence of hmac key with User-Info /6', async (t) => {
241271
const fastify = Fastify()
242272
await fastify.register(fastifySecureSession, { key, cookie: { path: '/', secure: false } })
243273

244-
await t.resolves(fastify.register(fastifyCsrf, {
245-
getUserInfo (req) {
246-
return req.session.get('username')
247-
},
248-
sessionPlugin: '@fastify/secure-session',
249-
csrfOpts: {
250-
hmacKey: 'foo'
251-
}
274+
await t.resolves(new Promise((resolve, reject) => {
275+
fastify.register(fastifyCsrf, {
276+
getUserInfo (req) {
277+
return req.session.get('username')
278+
},
279+
sessionPlugin: '@fastify/secure-session',
280+
csrfOpts: {
281+
hmacKey: 'foo'
282+
}
283+
}).then(() => {
284+
resolve()
285+
}).catch(err => {
286+
reject(err)
287+
})
252288
}))
253289
})

0 commit comments

Comments
 (0)