-
Notifications
You must be signed in to change notification settings - Fork 224
/
Copy pathcookieParser.js
302 lines (257 loc) · 10.1 KB
/
cookieParser.js
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
var assert = require('assert')
var cookieParser = require('..')
var http = require('http')
var request = require('supertest')
var signature = require('cookie-signature')
describe('cookieParser()', function () {
it('should export JSONCookies function', function () {
assert(typeof cookieParser.JSONCookies, 'function')
})
describe('when no cookies are sent', function () {
it('should default req.cookies to {}', function (done) {
request(createServer('keyboard cat'))
.get('/')
.expect(200, '{}', done)
})
it('should default req.signedCookies to {}', function (done) {
request(createServer('keyboard cat'))
.get('/signed')
.expect(200, '{}', done)
})
})
describe('when cookies are sent', function () {
it('should populate req.cookies', function (done) {
request(createServer('keyboard cat'))
.get('/')
.set('Cookie', 'foo=bar; bar=baz')
.expect(200, '{"foo":"bar","bar":"baz"}', done)
})
it('should inflate JSON cookies', function (done) {
request(createServer('keyboard cat'))
.get('/')
.set('Cookie', 'foo=j:{"foo":"bar"}')
.expect(200, '{"foo":{"foo":"bar"}}', done)
})
it('should not inflate invalid JSON cookies', function (done) {
request(createServer('keyboard cat'))
.get('/')
.set('Cookie', 'foo=j:{"foo":')
.expect(200, '{"foo":"j:{\\"foo\\":"}', done)
})
})
describe('when req.cookies exists', function () {
it('should do nothing', function (done) {
var _parser = cookieParser()
var server = http.createServer(function (req, res) {
req.cookies = { fizz: 'buzz' }
_parser(req, res, function (err) {
if (err) {
res.statusCode = 500
res.end(err.message)
return
}
res.end(JSON.stringify(req.cookies))
})
})
request(server)
.get('/')
.set('Cookie', 'foo=bar; bar=baz')
.expect(200, '{"fizz":"buzz"}', done)
})
})
describe('when a secret is given', function () {
var val = signature.sign('foobarbaz', 'keyboard cat')
// TODO: "bar" fails...
it('should populate req.signedCookies', function (done) {
request(createServer('keyboard cat'))
.get('/signed')
.set('Cookie', 'foo=s:' + val)
.expect(200, '{"foo":"foobarbaz"}', done)
})
it('should remove the signed value from req.cookies', function (done) {
request(createServer('keyboard cat'))
.get('/')
.set('Cookie', 'foo=s:' + val)
.expect(200, '{}', done)
})
it('should omit invalid signatures', function (done) {
var server = createServer('keyboard cat')
request(server)
.get('/signed')
.set('Cookie', 'foo=' + val + '3')
.expect(200, '{}', function (err) {
if (err) return done(err)
request(server)
.get('/')
.set('Cookie', 'foo=' + val + '3')
.expect(200, '{"foo":"foobarbaz.CP7AWaXDfAKIRfH49dQzKJx7sKzzSoPq7/AcBBRVwlI3"}', done)
})
})
})
describe('when multiple secrets are given', function () {
it('should populate req.signedCookies', function (done) {
request(createServer(['keyboard cat', 'nyan cat']))
.get('/signed')
.set('Cookie', 'buzz=s:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE; fizz=s:foobar.JTCAgiMWsnuZpN3mrYnEUjXlGxmDi4POCBnWbRxse88')
.expect(200, '{"buzz":"foobar","fizz":"foobar"}', done)
})
})
describe('when no secret is given', function () {
var server
before(function () {
server = createServer()
})
it('should populate req.cookies', function (done) {
request(server)
.get('/')
.set('Cookie', 'foo=bar; bar=baz')
.expect(200, '{"foo":"bar","bar":"baz"}', done)
})
it('should not populate req.signedCookies', function (done) {
var val = signature.sign('foobarbaz', 'keyboard cat')
request(server)
.get('/signed')
.set('Cookie', 'foo=s:' + val)
.expect(200, '{}', done)
})
})
})
describe('cookieParser.JSONCookie(str)', function () {
it('should return undefined for non-string arguments', function () {
assert.strictEqual(cookieParser.JSONCookie(), undefined)
assert.strictEqual(cookieParser.JSONCookie(undefined), undefined)
assert.strictEqual(cookieParser.JSONCookie(null), undefined)
assert.strictEqual(cookieParser.JSONCookie(42), undefined)
assert.strictEqual(cookieParser.JSONCookie({}), undefined)
assert.strictEqual(cookieParser.JSONCookie([]), undefined)
assert.strictEqual(cookieParser.JSONCookie(function () {}), undefined)
})
it('should return undefined for non-JSON cookie string', function () {
assert.strictEqual(cookieParser.JSONCookie(''), undefined)
assert.strictEqual(cookieParser.JSONCookie('foo'), undefined)
assert.strictEqual(cookieParser.JSONCookie('{}'), undefined)
})
it('should return object for JSON cookie string', function () {
assert.deepEqual(cookieParser.JSONCookie('j:{"foo":"bar"}'), { foo: 'bar' })
})
it('should return undefined on invalid JSON', function () {
assert.strictEqual(cookieParser.JSONCookie('j:{foo:"bar"}'), undefined)
})
})
describe('cookieParser.signedCookie(str, secret)', function () {
it('should return undefined for non-string arguments', function () {
assert.strictEqual(cookieParser.signedCookie(undefined, 'keyboard cat'), undefined)
assert.strictEqual(cookieParser.signedCookie(null, 'keyboard cat'), undefined)
assert.strictEqual(cookieParser.signedCookie(42, 'keyboard cat'), undefined)
assert.strictEqual(cookieParser.signedCookie({}, 'keyboard cat'), undefined)
assert.strictEqual(cookieParser.signedCookie([], 'keyboard cat'), undefined)
assert.strictEqual(cookieParser.signedCookie(function () {}, 'keyboard cat'), undefined)
})
it('should pass through non-signed string', function () {
assert.strictEqual(cookieParser.signedCookie('', 'keyboard cat'), '')
assert.strictEqual(cookieParser.signedCookie('foo', 'keyboard cat'), 'foo')
assert.strictEqual(cookieParser.signedCookie('j:{}', 'keyboard cat'), 'j:{}')
})
it('should return false for tampered signed string', function () {
assert.strictEqual(cookieParser.signedCookie('s:foobaz.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE', 'keyboard cat'), false)
})
it('should return unsigned value for signed string', function () {
assert.strictEqual(cookieParser.signedCookie('s:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE', 'keyboard cat'), 'foobar')
})
describe('when secret is an array', function () {
it('should return false for tampered signed string', function () {
assert.strictEqual(cookieParser.signedCookie('s:foobaz.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE', [
'keyboard cat',
'nyan cat'
]), false)
})
it('should return unsigned value for first secret', function () {
assert.strictEqual(cookieParser.signedCookie('s:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE', [
'keyboard cat',
'nyan cat'
]), 'foobar')
})
it('should return unsigned value for second secret', function () {
assert.strictEqual(cookieParser.signedCookie('s:foobar.JTCAgiMWsnuZpN3mrYnEUjXlGxmDi4POCBnWbRxse88', [
'keyboard cat',
'nyan cat'
]), 'foobar')
})
})
})
describe('cookieParser.signedCookies(obj, secret)', function () {
it('should ignore non-signed strings', function () {
assert.deepEqual(cookieParser.signedCookies({}, 'keyboard cat'), {})
assert.deepEqual(cookieParser.signedCookies({ foo: 'bar' }, 'keyboard cat'), {})
})
it('should include tampered strings as false', function () {
assert.deepEqual(cookieParser.signedCookies({ foo: 's:foobaz.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE' }, 'keyboard cat'), {
foo: false
})
})
it('should include unsigned strings', function () {
assert.deepEqual(cookieParser.signedCookies({ foo: 's:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE' }, 'keyboard cat'), {
foo: 'foobar'
})
})
it('should remove signed strings from original object', function () {
var obj = {
foo: 's:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE'
}
assert.deepEqual(cookieParser.signedCookies(obj, 'keyboard cat'), { foo: 'foobar' })
assert.deepEqual(obj, {})
})
it('should remove tampered strings from original object', function () {
var obj = {
foo: 's:foobaz.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE'
}
assert.deepEqual(cookieParser.signedCookies(obj, 'keyboard cat'), { foo: false })
assert.deepEqual(obj, {})
})
it('should leave unsigned string in original object', function () {
var obj = {
fizz: 'buzz',
foo: 's:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE'
}
assert.deepEqual(cookieParser.signedCookies(obj, 'keyboard cat'), { foo: 'foobar' })
assert.deepEqual(obj, { fizz: 'buzz' })
})
describe('when secret is an array', function () {
it('should include unsigned strings for matching secrets', function () {
var obj = {
buzz: 's:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE',
fizz: 's:foobar.JTCAgiMWsnuZpN3mrYnEUjXlGxmDi4POCBnWbRxse88'
}
assert.deepEqual(cookieParser.signedCookies(obj, ['keyboard cat']), {
buzz: 'foobar',
fizz: false
})
})
it('should include unsigned strings for all secrets', function () {
var obj = {
buzz: 's:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE',
fizz: 's:foobar.JTCAgiMWsnuZpN3mrYnEUjXlGxmDi4POCBnWbRxse88'
}
assert.deepEqual(cookieParser.signedCookies(obj, ['keyboard cat', 'nyan cat']), {
buzz: 'foobar',
fizz: 'foobar'
})
})
})
})
function createServer (secret) {
var _parser = cookieParser(secret)
return http.createServer(function (req, res) {
_parser(req, res, function (err) {
if (err) {
res.statusCode = 500
res.end(err.message)
return
}
var cookies = req.url === '/signed'
? req.signedCookies
: req.cookies
res.end(JSON.stringify(cookies))
})
})
}