@contrast/assess 1.46.1 → 1.46.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/package.json +14 -11
- package/lib/crypto-analysis/install/crypto.test.js +0 -146
- package/lib/crypto-analysis/install/math.test.js +0 -65
- package/lib/dataflow/index.test.js +0 -36
- package/lib/dataflow/propagation/index.test.js +0 -103
- package/lib/dataflow/propagation/install/JSON/index.test.js +0 -50
- package/lib/dataflow/propagation/install/JSON/parse-fn.test.js +0 -232
- package/lib/dataflow/propagation/install/JSON/parse.test.js +0 -968
- package/lib/dataflow/propagation/install/JSON/stringify.test.js +0 -265
- package/lib/dataflow/propagation/install/array-prototype-join.test.js +0 -106
- package/lib/dataflow/propagation/install/buffer.test.js +0 -112
- package/lib/dataflow/propagation/install/contrast-methods/add.test.js +0 -94
- package/lib/dataflow/propagation/install/contrast-methods/index.test.js +0 -49
- package/lib/dataflow/propagation/install/contrast-methods/number.test.js +0 -50
- package/lib/dataflow/propagation/install/contrast-methods/string.test.js +0 -148
- package/lib/dataflow/propagation/install/contrast-methods/tag.test.js +0 -145
- package/lib/dataflow/propagation/install/decode-uri-component.test.js +0 -78
- package/lib/dataflow/propagation/install/ejs/escape-xml.test.js +0 -69
- package/lib/dataflow/propagation/install/ejs/template.test.js +0 -62
- package/lib/dataflow/propagation/install/encode-uri.test.js +0 -83
- package/lib/dataflow/propagation/install/escape-html.test.js +0 -71
- package/lib/dataflow/propagation/install/escape.test.js +0 -73
- package/lib/dataflow/propagation/install/fastify-send.test.js +0 -42
- package/lib/dataflow/propagation/install/handlebars-utils-escape-expression.test.js +0 -71
- package/lib/dataflow/propagation/install/isnumeric-0.test.js +0 -58
- package/lib/dataflow/propagation/install/joi/any.test.js +0 -270
- package/lib/dataflow/propagation/install/joi/array.test.js +0 -912
- package/lib/dataflow/propagation/install/joi/boolean.test.js +0 -103
- package/lib/dataflow/propagation/install/joi/expression.test.js +0 -76
- package/lib/dataflow/propagation/install/joi/index.test.js +0 -39
- package/lib/dataflow/propagation/install/joi/number.test.js +0 -103
- package/lib/dataflow/propagation/install/joi/object.test.js +0 -119
- package/lib/dataflow/propagation/install/joi/ref.test.js +0 -607
- package/lib/dataflow/propagation/install/joi/string-schema.test.js +0 -513
- package/lib/dataflow/propagation/install/mongoose/index.test.js +0 -42
- package/lib/dataflow/propagation/install/mongoose/schema-map.test.js +0 -348
- package/lib/dataflow/propagation/install/mongoose/schema-mixed.test.js +0 -511
- package/lib/dataflow/propagation/install/mongoose/schema-string.test.js +0 -199
- package/lib/dataflow/propagation/install/mustache-escape.test.js +0 -62
- package/lib/dataflow/propagation/install/mysql-connection-escape.test.js +0 -74
- package/lib/dataflow/propagation/install/parse-int.test.js +0 -48
- package/lib/dataflow/propagation/install/path/basename.test.js +0 -143
- package/lib/dataflow/propagation/install/path/dirname.test.js +0 -167
- package/lib/dataflow/propagation/install/path/extname.test.js +0 -141
- package/lib/dataflow/propagation/install/path/format.test.js +0 -250
- package/lib/dataflow/propagation/install/path/index.test.js +0 -45
- package/lib/dataflow/propagation/install/path/join-and-resolve.test.js +0 -485
- package/lib/dataflow/propagation/install/path/normalize.test.js +0 -176
- package/lib/dataflow/propagation/install/path/parse.test.js +0 -238
- package/lib/dataflow/propagation/install/path/relative.test.js +0 -239
- package/lib/dataflow/propagation/install/path/toNamespacedPath.test.js +0 -158
- package/lib/dataflow/propagation/install/pug/index.test.js +0 -55
- package/lib/dataflow/propagation/install/pug-runtime-escape.test.js +0 -69
- package/lib/dataflow/propagation/install/querystring/escape.test.js +0 -63
- package/lib/dataflow/propagation/install/querystring/index.test.js +0 -40
- package/lib/dataflow/propagation/install/querystring/parse.test.js +0 -272
- package/lib/dataflow/propagation/install/querystring/stringify.test.js +0 -301
- package/lib/dataflow/propagation/install/reg-exp-prototype-exec.test.js +0 -283
- package/lib/dataflow/propagation/install/send.test.js +0 -63
- package/lib/dataflow/propagation/install/sequelize/query-generator.test.js +0 -74
- package/lib/dataflow/propagation/install/sequelize/sql-string.test.js +0 -119
- package/lib/dataflow/propagation/install/sql-template-strings.test.js +0 -100
- package/lib/dataflow/propagation/install/string/concat.test.js +0 -145
- package/lib/dataflow/propagation/install/string/format-methods.test.js +0 -74
- package/lib/dataflow/propagation/install/string/html-methods.test.js +0 -177
- package/lib/dataflow/propagation/install/string/index.test.js +0 -103
- package/lib/dataflow/propagation/install/string/match-all.test.js +0 -412
- package/lib/dataflow/propagation/install/string/match.test.js +0 -374
- package/lib/dataflow/propagation/install/string/replace.test.js +0 -601
- package/lib/dataflow/propagation/install/string/slice.test.js +0 -278
- package/lib/dataflow/propagation/install/string/split.test.js +0 -513
- package/lib/dataflow/propagation/install/string/substring.test.js +0 -251
- package/lib/dataflow/propagation/install/string/trim.test.js +0 -135
- package/lib/dataflow/propagation/install/unescape.test.js +0 -78
- package/lib/dataflow/propagation/install/url/domain-parsers.test.js +0 -63
- package/lib/dataflow/propagation/install/url/parse.test.js +0 -391
- package/lib/dataflow/propagation/install/url/searchParams.test.js +0 -538
- package/lib/dataflow/propagation/install/url/url.test.js +0 -466
- package/lib/dataflow/propagation/install/util-format.test.js +0 -336
- package/lib/dataflow/propagation/install/validator/hooks.test.js +0 -211
- package/lib/dataflow/sinks/index.test.js +0 -78
- package/lib/dataflow/sinks/install/child-process.test.js +0 -338
- package/lib/dataflow/sinks/install/eval.test.js +0 -95
- package/lib/dataflow/sinks/install/express/index.test.js +0 -33
- package/lib/dataflow/sinks/install/express/reflected-xss.test.js +0 -109
- package/lib/dataflow/sinks/install/express/unvalidated-redirect.test.js +0 -144
- package/lib/dataflow/sinks/install/fastify/index.test.js +0 -32
- package/lib/dataflow/sinks/install/fastify/unvalidated-redirect.test.js +0 -130
- package/lib/dataflow/sinks/install/fs.test.js +0 -138
- package/lib/dataflow/sinks/install/function.test.js +0 -103
- package/lib/dataflow/sinks/install/hapi/index.test.js +0 -32
- package/lib/dataflow/sinks/install/hapi/unvalidated-redirect.test.js +0 -130
- package/lib/dataflow/sinks/install/http/index.test.js +0 -33
- package/lib/dataflow/sinks/install/http/request.test.js +0 -184
- package/lib/dataflow/sinks/install/http/server-response.test.js +0 -160
- package/lib/dataflow/sinks/install/koa/index.test.js +0 -32
- package/lib/dataflow/sinks/install/koa/unvalidated-redirect.test.js +0 -200
- package/lib/dataflow/sinks/install/libxmljs.test.js +0 -158
- package/lib/dataflow/sinks/install/marsdb.test.js +0 -166
- package/lib/dataflow/sinks/install/mongodb.test.js +0 -617
- package/lib/dataflow/sinks/install/mssql.test.js +0 -134
- package/lib/dataflow/sinks/install/mysql.test.js +0 -224
- package/lib/dataflow/sinks/install/node-serialize.test.js +0 -83
- package/lib/dataflow/sinks/install/postgres.test.js +0 -152
- package/lib/dataflow/sinks/install/restify.test.js +0 -140
- package/lib/dataflow/sinks/install/sequelize.test.js +0 -100
- package/lib/dataflow/sinks/install/sqlite3.test.js +0 -118
- package/lib/dataflow/sinks/install/vm.test.js +0 -326
- package/lib/dataflow/sources/handler.test.js +0 -501
- package/lib/dataflow/sources/index.test.js +0 -60
- package/lib/dataflow/sources/install/body-parser1.test.js +0 -244
- package/lib/dataflow/sources/install/busboy.test.js +0 -152
- package/lib/dataflow/sources/install/cookie-parser1.test.js +0 -141
- package/lib/dataflow/sources/install/express/params.test.js +0 -110
- package/lib/dataflow/sources/install/express/parsedUrl.test.js +0 -107
- package/lib/dataflow/sources/install/fastify/fastify.test.js +0 -207
- package/lib/dataflow/sources/install/fastify/index.test.js +0 -33
- package/lib/dataflow/sources/install/formidable1.test.js +0 -119
- package/lib/dataflow/sources/install/graphql-http.test.js +0 -133
- package/lib/dataflow/sources/install/hapi/hapi.test.js +0 -171
- package/lib/dataflow/sources/install/hapi/index.test.js +0 -33
- package/lib/dataflow/sources/install/http.test.js +0 -149
- package/lib/dataflow/sources/install/koa/index.test.js +0 -40
- package/lib/dataflow/sources/install/koa/koa-bodyparsers.test.js +0 -160
- package/lib/dataflow/sources/install/koa/koa-multer.test.js +0 -197
- package/lib/dataflow/sources/install/koa/koa-routers.test.js +0 -146
- package/lib/dataflow/sources/install/koa/koa2.test.js +0 -145
- package/lib/dataflow/sources/install/multer1.test.js +0 -143
- package/lib/dataflow/sources/install/qs6.test.js +0 -131
- package/lib/dataflow/sources/install/querystring.test.js +0 -82
- package/lib/dataflow/sources/install/restify/fieldedTextBodyParser.test.js +0 -86
- package/lib/dataflow/sources/install/restify/index.test.js +0 -38
- package/lib/dataflow/sources/install/restify/jsonBodyParser.test.js +0 -140
- package/lib/dataflow/sources/install/restify/router.test.js +0 -81
- package/lib/dataflow/tag-utils-complete.test.js +0 -27
- package/lib/dataflow/tag-utils.test.js +0 -192
- package/lib/dataflow/tracker.test.js +0 -216
- package/lib/dataflow/utils/is-safe-content-type.test.js +0 -16
- package/lib/dataflow/utils/is-vulnerable.test.js +0 -115
- package/lib/event-factory.test.js +0 -326
- package/lib/get-policy.test.js +0 -194
- package/lib/get-source-context.test.js +0 -161
- package/lib/index.test.js +0 -45
- package/lib/make-source-context.test.js +0 -50
- package/lib/response-scanning/handlers/index.test.js +0 -419
- package/lib/response-scanning/handlers/utils.test.js +0 -380
- package/lib/response-scanning/index.test.js +0 -41
- package/lib/response-scanning/install/http.test.js +0 -175
- package/lib/rule-scopes.test.js +0 -27
- package/lib/sampler/common.test.js +0 -101
- package/lib/sampler/index.test.js +0 -313
- package/lib/session-configuration/handlers.test.js +0 -84
- package/lib/session-configuration/index.test.js +0 -36
- package/lib/session-configuration/install/express-session.test.js +0 -218
- package/lib/session-configuration/install/fastify-cookie.test.js +0 -63
- package/lib/session-configuration/install/hapi.test.js +0 -269
- package/lib/session-configuration/install/koa.test.js +0 -92
|
@@ -1,513 +0,0 @@
|
|
|
1
|
-
'use strict';
|
|
2
|
-
|
|
3
|
-
const { expect } = require('chai');
|
|
4
|
-
const { initAssessFixture } = require('@contrast/test/fixtures');
|
|
5
|
-
const { DataflowTag } = require('@contrast/common');
|
|
6
|
-
|
|
7
|
-
describe('assess dataflow propagation joi string', function() {
|
|
8
|
-
let core, joi, tracker, trackString, simulateRequestScope, strInstr;
|
|
9
|
-
|
|
10
|
-
beforeEach(function() {
|
|
11
|
-
({ core, simulateRequestScope, trackString } = initAssessFixture());
|
|
12
|
-
tracker = core.assess.dataflow.tracker;
|
|
13
|
-
|
|
14
|
-
core.depHooks.resolve.withArgs({ name: 'joi', file: 'lib/types/string.js', version: '>=17 <18' }).yields(require('joi-17/lib/types/string'));
|
|
15
|
-
core.depHooks.resolve.withArgs({ name: 'joi', file: 'lib/types/keys.js', version: '>=17 <18' }).yields(require('joi-17/lib/types/keys'));
|
|
16
|
-
core.depHooks.resolve.withArgs({ name: 'joi', file: 'lib/validator', version: '>=17 <18' }).yields(require('joi-17/lib/validator'));
|
|
17
|
-
core.depHooks.resolve.withArgs({ name: 'joi', file: 'lib/values.js', version: '>=17 <18' }).yields(require('joi-17/lib/values'));
|
|
18
|
-
|
|
19
|
-
require('./index')(core).install();
|
|
20
|
-
strInstr = require('../string')(core);
|
|
21
|
-
strInstr.install();
|
|
22
|
-
joi = require('joi-17');
|
|
23
|
-
});
|
|
24
|
-
|
|
25
|
-
afterEach(function() {
|
|
26
|
-
Object.keys(require.cache).forEach((key) => {
|
|
27
|
-
if (key.includes('joi')) {
|
|
28
|
-
delete require.cache[key];
|
|
29
|
-
}
|
|
30
|
-
});
|
|
31
|
-
strInstr.uninstall();
|
|
32
|
-
});
|
|
33
|
-
|
|
34
|
-
describe('Joi string-schema validator tests', function() {
|
|
35
|
-
const stringValidatorExamples = [
|
|
36
|
-
{
|
|
37
|
-
ruleNames: ['base64'],
|
|
38
|
-
validInput: 'VE9PTUFOWVNFQ1JFVFM=',
|
|
39
|
-
invalidInput: 'VE9PTUFOWVNFQ1JFVFM',
|
|
40
|
-
expectedTags: DataflowTag.ALPHANUM_SPACE_HYPHEN,
|
|
41
|
-
},
|
|
42
|
-
{
|
|
43
|
-
ruleNames: ['guid'],
|
|
44
|
-
validInput: '8594cc30-171E-4a21-99ce-BC0DC55EDEE8',
|
|
45
|
-
invalidInput: '1',
|
|
46
|
-
expectedTags: DataflowTag.ALPHANUM_SPACE_HYPHEN,
|
|
47
|
-
},
|
|
48
|
-
{
|
|
49
|
-
ruleNames: ['alphanum'],
|
|
50
|
-
validInput: 'abc123ABC',
|
|
51
|
-
invalidInput: '_&*"a',
|
|
52
|
-
expectedTags: DataflowTag.ALPHANUM_SPACE_HYPHEN,
|
|
53
|
-
},
|
|
54
|
-
{
|
|
55
|
-
ruleNames: ['hex'],
|
|
56
|
-
validInput: '54455354',
|
|
57
|
-
invalidInput: 'z',
|
|
58
|
-
expectedTags: DataflowTag.ALPHANUM_SPACE_HYPHEN,
|
|
59
|
-
},
|
|
60
|
-
{
|
|
61
|
-
ruleNames: ['isoDate'],
|
|
62
|
-
validInput: '2018-11-28T18:25:32.000Z',
|
|
63
|
-
invalidInput: '2018-28-11',
|
|
64
|
-
expectedTags: DataflowTag.ALPHANUM_SPACE_HYPHEN,
|
|
65
|
-
},
|
|
66
|
-
{
|
|
67
|
-
ruleNames: ['isoDuration'],
|
|
68
|
-
validInput: 'P3Y6M4DT12H30M5S',
|
|
69
|
-
invalidInput: '2020/10/03',
|
|
70
|
-
expectedTags: DataflowTag.ALPHANUM_SPACE_HYPHEN,
|
|
71
|
-
},
|
|
72
|
-
{
|
|
73
|
-
ruleNames: ['isoDuration'],
|
|
74
|
-
validInput: 'P3Y6M4DT12H30M5S',
|
|
75
|
-
invalidInput: 'P2020-08-06T12:30:00', // valid ISO 8061 format, but not supported in Joi
|
|
76
|
-
expectedTags: DataflowTag.ALPHANUM_SPACE_HYPHEN,
|
|
77
|
-
},
|
|
78
|
-
{
|
|
79
|
-
ruleNames: ['creditCard'],
|
|
80
|
-
validInput: '4111111111111111',
|
|
81
|
-
invalidInput: '0000000000000000',
|
|
82
|
-
expectedTags: DataflowTag.LIMITED_CHARS,
|
|
83
|
-
},
|
|
84
|
-
{
|
|
85
|
-
ruleNames: ['ip'],
|
|
86
|
-
validInput: '0.0.0.0',
|
|
87
|
-
invalidInput: '256.258.301.1',
|
|
88
|
-
expectedTags: DataflowTag.LIMITED_CHARS,
|
|
89
|
-
},
|
|
90
|
-
{
|
|
91
|
-
ruleNames: ['token'],
|
|
92
|
-
validInput: '_azAZ09',
|
|
93
|
-
invalidInput: '(-.-)Zzz...',
|
|
94
|
-
expectedTags: DataflowTag.ALPHANUM_SPACE_HYPHEN,
|
|
95
|
-
},
|
|
96
|
-
{
|
|
97
|
-
ruleNames: ['hostname'],
|
|
98
|
-
validInput: 'www.contrastsecurity.com',
|
|
99
|
-
invalidInput: 'localhost:8000',
|
|
100
|
-
expectedTags: DataflowTag.ALPHANUM_SPACE_HYPHEN,
|
|
101
|
-
},
|
|
102
|
-
{
|
|
103
|
-
ruleNames: ['domain'],
|
|
104
|
-
validInput: 'www.contrastsecurity.com',
|
|
105
|
-
invalidInput: 'localhost:8000',
|
|
106
|
-
expectedTags: DataflowTag.ALPHANUM_SPACE_HYPHEN,
|
|
107
|
-
},
|
|
108
|
-
];
|
|
109
|
-
|
|
110
|
-
// Testing individually rules and expected tags here, but keeping array structure for consistency
|
|
111
|
-
stringValidatorExamples.forEach(
|
|
112
|
-
({ validInput, invalidInput = 'foo', expectedTags, ruleNames }) => {
|
|
113
|
-
const methodNames = ruleNames.map((n) => `${n}()`).join('.');
|
|
114
|
-
|
|
115
|
-
it(`tracked values passing validation for ${methodNames} will be tagged with ${expectedTags}`, function(done) {
|
|
116
|
-
simulateRequestScope(() => {
|
|
117
|
-
const tracked = trackString(validInput);
|
|
118
|
-
let schema = joi.string();
|
|
119
|
-
schema = schema[ruleNames[0]]();
|
|
120
|
-
|
|
121
|
-
// One check with validateAsync to make sure it works too
|
|
122
|
-
schema.validateAsync(tracked)
|
|
123
|
-
.then((validated) => {
|
|
124
|
-
const data = tracker.getData(validated);
|
|
125
|
-
|
|
126
|
-
expect(validInput).to.equal(validated);
|
|
127
|
-
|
|
128
|
-
expect(data.tags).to.deep.equal({
|
|
129
|
-
[DataflowTag.UNTRUSTED]: [0, validInput.length - 1],
|
|
130
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, validInput.length - 1],
|
|
131
|
-
[expectedTags]: [0, validInput.length - 1]
|
|
132
|
-
});
|
|
133
|
-
|
|
134
|
-
expect(data.moduleName).to.equal('joi');
|
|
135
|
-
expect(data.methodName).to.equal(`string.${ruleNames[0]}`);
|
|
136
|
-
done();
|
|
137
|
-
})
|
|
138
|
-
.catch(err => done(err));
|
|
139
|
-
});
|
|
140
|
-
});
|
|
141
|
-
|
|
142
|
-
it(`tracked values for ${methodNames} should not be tagged if not validated`, function() {
|
|
143
|
-
simulateRequestScope(() => {
|
|
144
|
-
const tracked = trackString(invalidInput);
|
|
145
|
-
let schema = joi.string();
|
|
146
|
-
schema = schema[ruleNames[0]]();
|
|
147
|
-
|
|
148
|
-
const validated = schema.validate(tracked);
|
|
149
|
-
const data = tracker.getData(validated.value);
|
|
150
|
-
|
|
151
|
-
expect(data.tags).to.deep.equal({
|
|
152
|
-
[DataflowTag.UNTRUSTED]: [0, invalidInput.length - 1],
|
|
153
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, invalidInput.length - 1],
|
|
154
|
-
});
|
|
155
|
-
});
|
|
156
|
-
});
|
|
157
|
-
|
|
158
|
-
it(`should not track the result of ${methodNames} if value is not tracked`, function() {
|
|
159
|
-
simulateRequestScope(() => {
|
|
160
|
-
let schema = joi.string();
|
|
161
|
-
schema = schema[ruleNames[0]]();
|
|
162
|
-
|
|
163
|
-
const validated = schema.validate(validInput);
|
|
164
|
-
const data = tracker.getData(validated.value);
|
|
165
|
-
|
|
166
|
-
expect(data).to.be.null;
|
|
167
|
-
});
|
|
168
|
-
});
|
|
169
|
-
}
|
|
170
|
-
);
|
|
171
|
-
|
|
172
|
-
['custom', 'external'].forEach((method) => {
|
|
173
|
-
it(`should custom validate the result when ${method} validation passes with the same value`, function(done) {
|
|
174
|
-
simulateRequestScope(() => {
|
|
175
|
-
core.config.assess.trust_custom_validators = true;
|
|
176
|
-
const joiSchema = joi.string()[method](() => trackString('1234'));
|
|
177
|
-
|
|
178
|
-
const value = trackString('1234');
|
|
179
|
-
|
|
180
|
-
joiSchema.validateAsync(value)
|
|
181
|
-
.then((data) => {
|
|
182
|
-
const argInfo = tracker.getData(value);
|
|
183
|
-
const resInfo = tracker.getData(data);
|
|
184
|
-
expect(argInfo.methodName).to.include(`string.${method}`);
|
|
185
|
-
expect(argInfo.tags).to.deep.equal({
|
|
186
|
-
[DataflowTag.UNTRUSTED]: [0, 3],
|
|
187
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, 3],
|
|
188
|
-
[DataflowTag.CUSTOM_VALIDATED]: [0, 3],
|
|
189
|
-
});
|
|
190
|
-
// with .external() the result is the original
|
|
191
|
-
// arg string instance when they are equal
|
|
192
|
-
if (method === 'external') {
|
|
193
|
-
expect(tracker.getData(data)).to.equal(argInfo);
|
|
194
|
-
} else {
|
|
195
|
-
expect(resInfo.methodName).to.include(`string.${method}`);
|
|
196
|
-
expect(resInfo.tags).to.deep.equal({
|
|
197
|
-
[DataflowTag.UNTRUSTED]: [0, 3],
|
|
198
|
-
[DataflowTag.CUSTOM_VALIDATED]: [0, 3],
|
|
199
|
-
});
|
|
200
|
-
}
|
|
201
|
-
done();
|
|
202
|
-
})
|
|
203
|
-
.catch(err => done(err));
|
|
204
|
-
});
|
|
205
|
-
});
|
|
206
|
-
|
|
207
|
-
it(`should custom validate only the result if it passes ${method} validation with a different value`, function(done) {
|
|
208
|
-
simulateRequestScope(() => {
|
|
209
|
-
core.config.assess.trust_custom_validators = true;
|
|
210
|
-
const joiSchema = joi.string()[method](() => trackString('5'));
|
|
211
|
-
|
|
212
|
-
const value = trackString('1234');
|
|
213
|
-
|
|
214
|
-
joiSchema.validateAsync(value)
|
|
215
|
-
.then((result) => {
|
|
216
|
-
const argInfo = tracker.getData(value);
|
|
217
|
-
const resultInfo = tracker.getData(result);
|
|
218
|
-
|
|
219
|
-
expect(argInfo.methodName).to.equal('string.validate');
|
|
220
|
-
expect(argInfo.tags).to.deep.equal({
|
|
221
|
-
[DataflowTag.UNTRUSTED]: [0, 3],
|
|
222
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, 3],
|
|
223
|
-
});
|
|
224
|
-
expect(resultInfo.methodName).to.include(`string.${method}`);
|
|
225
|
-
expect(resultInfo.tags).to.deep.equal({
|
|
226
|
-
[DataflowTag.UNTRUSTED]: [0, 0],
|
|
227
|
-
[DataflowTag.CUSTOM_VALIDATED]: [0, 0],
|
|
228
|
-
});
|
|
229
|
-
done();
|
|
230
|
-
})
|
|
231
|
-
.catch(err => done(err));
|
|
232
|
-
});
|
|
233
|
-
});
|
|
234
|
-
|
|
235
|
-
it('should not propagate if the result is not tracked and different', function(done) {
|
|
236
|
-
simulateRequestScope(() => {
|
|
237
|
-
core.config.assess.trust_custom_validators = true;
|
|
238
|
-
const joiSchema = joi.string()[method](() => '5');
|
|
239
|
-
|
|
240
|
-
const value = trackString('1234');
|
|
241
|
-
|
|
242
|
-
joiSchema.validateAsync(value)
|
|
243
|
-
.then((result) => {
|
|
244
|
-
const argInfo = tracker.getData(value);
|
|
245
|
-
const resultInfo = tracker.getData(result);
|
|
246
|
-
|
|
247
|
-
expect(argInfo.methodName).to.equal('string.validate');
|
|
248
|
-
expect(argInfo.tags).to.deep.equal({
|
|
249
|
-
[DataflowTag.UNTRUSTED]: [0, 3],
|
|
250
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, 3],
|
|
251
|
-
});
|
|
252
|
-
expect(resultInfo).to.be.null;
|
|
253
|
-
done();
|
|
254
|
-
})
|
|
255
|
-
.catch(err => done(err));
|
|
256
|
-
});
|
|
257
|
-
});
|
|
258
|
-
});
|
|
259
|
-
|
|
260
|
-
const stringCoercingExamples = [
|
|
261
|
-
{
|
|
262
|
-
ruleNames: ['lowercase'],
|
|
263
|
-
validInput: 'lorem ipsum',
|
|
264
|
-
invalidInput: 'Lorem Ipsum',
|
|
265
|
-
expectedTags: DataflowTag.UNTRUSTED,
|
|
266
|
-
},
|
|
267
|
-
{
|
|
268
|
-
ruleNames: ['uppercase'],
|
|
269
|
-
validInput: 'LOREM IPSUM',
|
|
270
|
-
invalidInput: 'Lorem Ipsum',
|
|
271
|
-
expectedTags: DataflowTag.UNTRUSTED,
|
|
272
|
-
},
|
|
273
|
-
{
|
|
274
|
-
ruleNames: ['isoDate'],
|
|
275
|
-
validInput: '2018-11-28T18:25:32.000Z',
|
|
276
|
-
invalidInput: '2018-28-11',
|
|
277
|
-
expectedTags: DataflowTag.ALPHANUM_SPACE_HYPHEN,
|
|
278
|
-
},
|
|
279
|
-
];
|
|
280
|
-
|
|
281
|
-
stringCoercingExamples.forEach(
|
|
282
|
-
({ validInput, invalidInput = 'foo', expectedTags, ruleNames }) => {
|
|
283
|
-
const methodNames = ruleNames.map((n) => `${n}()`).join('.');
|
|
284
|
-
|
|
285
|
-
it(`tracked values passing validation for ${methodNames} with coercion will be tagged with ${expectedTags}`, function() {
|
|
286
|
-
simulateRequestScope(() => {
|
|
287
|
-
const tracked = trackString(validInput);
|
|
288
|
-
let schema = joi.string();
|
|
289
|
-
schema = schema[ruleNames[0]]();
|
|
290
|
-
|
|
291
|
-
const validated = schema.validate(tracked, { convert: true });
|
|
292
|
-
const data = tracker.getData(validated.value);
|
|
293
|
-
|
|
294
|
-
expect(validated.value).to.equal(validInput);
|
|
295
|
-
expect(data.tags).to.deep.equal({
|
|
296
|
-
[DataflowTag.UNTRUSTED]: [0, validInput.length - 1],
|
|
297
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, validInput.length - 1],
|
|
298
|
-
[expectedTags]: [0, validInput.length - 1]
|
|
299
|
-
});
|
|
300
|
-
});
|
|
301
|
-
});
|
|
302
|
-
|
|
303
|
-
it(`tracked values passing validation for ${methodNames} without coercion will be tagged with ${expectedTags}`, function() {
|
|
304
|
-
simulateRequestScope(() => {
|
|
305
|
-
const tracked = trackString(validInput);
|
|
306
|
-
let schema = joi.string();
|
|
307
|
-
schema = schema[ruleNames[0]]();
|
|
308
|
-
|
|
309
|
-
const validated = schema.validate(tracked, { convert: false });
|
|
310
|
-
const data = tracker.getData(validated.value);
|
|
311
|
-
|
|
312
|
-
expect(validInput).to.equal(validated.value);
|
|
313
|
-
expect(data.tags).to.deep.equal({
|
|
314
|
-
[DataflowTag.UNTRUSTED]: [0, validInput.length - 1],
|
|
315
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, validInput.length - 1],
|
|
316
|
-
[expectedTags]: [0, validInput.length - 1]
|
|
317
|
-
});
|
|
318
|
-
});
|
|
319
|
-
});
|
|
320
|
-
|
|
321
|
-
it(`tracked values for ${methodNames} without coercion should not be tagged if not valid`, function() {
|
|
322
|
-
simulateRequestScope(() => {
|
|
323
|
-
const tracked = trackString(invalidInput);
|
|
324
|
-
let schema = joi.string();
|
|
325
|
-
schema = schema[ruleNames[0]]();
|
|
326
|
-
|
|
327
|
-
const validated = schema.validate(tracked, { convert: false });
|
|
328
|
-
const data = tracker.getData(validated.value);
|
|
329
|
-
|
|
330
|
-
expect(data.tags).to.deep.equal({
|
|
331
|
-
[DataflowTag.UNTRUSTED]: [0, invalidInput.length - 1],
|
|
332
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, invalidInput.length - 1],
|
|
333
|
-
});
|
|
334
|
-
});
|
|
335
|
-
});
|
|
336
|
-
|
|
337
|
-
it(`should not track the result of ${methodNames} if input value is not tracked`, function() {
|
|
338
|
-
simulateRequestScope(() => {
|
|
339
|
-
const schema = joi.string();
|
|
340
|
-
|
|
341
|
-
const validated = schema.validate(validInput);
|
|
342
|
-
const data = tracker.getData(validated.value);
|
|
343
|
-
|
|
344
|
-
expect(data).to.be.null;
|
|
345
|
-
});
|
|
346
|
-
});
|
|
347
|
-
}
|
|
348
|
-
);
|
|
349
|
-
|
|
350
|
-
[
|
|
351
|
-
{
|
|
352
|
-
ruleNames: ['isoDate'],
|
|
353
|
-
validInput: '2018-11-28T18:25:32.000Z',
|
|
354
|
-
invalidInput: '2018-28-11',
|
|
355
|
-
},
|
|
356
|
-
{
|
|
357
|
-
ruleNames: ['isoDate'],
|
|
358
|
-
validInput: '2018-11-28T18:25:32',
|
|
359
|
-
invalidInput: '002013-06-07T14:21:46.295Z',
|
|
360
|
-
},
|
|
361
|
-
{
|
|
362
|
-
ruleNames: ['isoDate'],
|
|
363
|
-
validInput: '2013-06-07T14:21:46.295+07:00',
|
|
364
|
-
invalidInput: '+2013-06-07T14:21:46.295Z',
|
|
365
|
-
},
|
|
366
|
-
{
|
|
367
|
-
ruleNames: ['isoDate'],
|
|
368
|
-
validInput: '+002013-06-07T14:21:46.295Z',
|
|
369
|
-
invalidInput: '2018-28-11T',
|
|
370
|
-
},
|
|
371
|
-
{
|
|
372
|
-
ruleNames: ['isoDate'],
|
|
373
|
-
validInput: '2013-06-07T24:00',
|
|
374
|
-
invalidInput: '2013-06-07T24:21',
|
|
375
|
-
},
|
|
376
|
-
].forEach(({ validInput, invalidInput }) => {
|
|
377
|
-
it('should track isoDate when coercing', function() {
|
|
378
|
-
simulateRequestScope(() => {
|
|
379
|
-
const tracked = trackString(validInput);
|
|
380
|
-
let schema = joi.string();
|
|
381
|
-
schema = schema.isoDate();
|
|
382
|
-
|
|
383
|
-
const validated = schema.validate(tracked, { convert: true });
|
|
384
|
-
const data = tracker.getData(validated.value);
|
|
385
|
-
|
|
386
|
-
expect(data.tags).to.deep.equal({
|
|
387
|
-
[DataflowTag.UNTRUSTED]: [0, 23], // coerced isoDate has fixed size
|
|
388
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, 23],
|
|
389
|
-
[DataflowTag.ALPHANUM_SPACE_HYPHEN]: [0, 23]
|
|
390
|
-
});
|
|
391
|
-
});
|
|
392
|
-
});
|
|
393
|
-
|
|
394
|
-
it('should not track, nor tag isoDate values if not validated', function() {
|
|
395
|
-
simulateRequestScope(() => {
|
|
396
|
-
const tracked = trackString(invalidInput);
|
|
397
|
-
let schema = joi.string();
|
|
398
|
-
schema = schema.isoDate();
|
|
399
|
-
|
|
400
|
-
const validated = schema.validate(tracked);
|
|
401
|
-
const data = tracker.getData(validated.value);
|
|
402
|
-
|
|
403
|
-
expect(data.tags).to.deep.equal({
|
|
404
|
-
[DataflowTag.UNTRUSTED]: [0, validated.value.length - 1], // coerced isoDate has fixed size
|
|
405
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, validated.value.length - 1],
|
|
406
|
-
});
|
|
407
|
-
});
|
|
408
|
-
});
|
|
409
|
-
});
|
|
410
|
-
|
|
411
|
-
const chainedExamples = [
|
|
412
|
-
{
|
|
413
|
-
ruleNames: ['lowercase', 'guid'],
|
|
414
|
-
validInput: '8594cc30-171e-4a21-99ce-bc0dc55edee8',
|
|
415
|
-
invalidInput: '8594cc30',
|
|
416
|
-
expectedTags: [DataflowTag.ALPHANUM_SPACE_HYPHEN],
|
|
417
|
-
},
|
|
418
|
-
{
|
|
419
|
-
ruleNames: ['hex', 'alphanum'],
|
|
420
|
-
validInput: '8594cc30',
|
|
421
|
-
invalidInput: 'z~',
|
|
422
|
-
expectedTags: [DataflowTag.ALPHANUM_SPACE_HYPHEN],
|
|
423
|
-
},
|
|
424
|
-
{
|
|
425
|
-
ruleNames: ['creditCard', 'alphanum'],
|
|
426
|
-
validInput: '4111111111111111',
|
|
427
|
-
invalidInput: '0000000000000000',
|
|
428
|
-
expectedTags: [DataflowTag.LIMITED_CHARS, DataflowTag.ALPHANUM_SPACE_HYPHEN],
|
|
429
|
-
},
|
|
430
|
-
{
|
|
431
|
-
ruleNames: ['alphanum', 'creditCard'],
|
|
432
|
-
validInput: '4111111111111111',
|
|
433
|
-
invalidInput: '0000000000000000a',
|
|
434
|
-
expectedTags: [DataflowTag.LIMITED_CHARS, DataflowTag.ALPHANUM_SPACE_HYPHEN],
|
|
435
|
-
onFailureExpectedTags: [DataflowTag.ALPHANUM_SPACE_HYPHEN],
|
|
436
|
-
},
|
|
437
|
-
{
|
|
438
|
-
ruleNames: ['lowercase', 'token', 'alphanum'],
|
|
439
|
-
validInput: 'lowercharsonly',
|
|
440
|
-
invalidInput: 'valid_lowercase_alphanum_but_with_underscores',
|
|
441
|
-
expectedTags: [DataflowTag.ALPHANUM_SPACE_HYPHEN],
|
|
442
|
-
onFailureExpectedTags: [DataflowTag.ALPHANUM_SPACE_HYPHEN],
|
|
443
|
-
},
|
|
444
|
-
{
|
|
445
|
-
ruleNames: ['alphanum', 'lowercase', 'token'],
|
|
446
|
-
validInput: 'lowercharsonly',
|
|
447
|
-
invalidInput: 'valid_lowercase_alphanum_but_with_underscores',
|
|
448
|
-
expectedTags: [DataflowTag.ALPHANUM_SPACE_HYPHEN],
|
|
449
|
-
},
|
|
450
|
-
];
|
|
451
|
-
|
|
452
|
-
chainedExamples.forEach(
|
|
453
|
-
({
|
|
454
|
-
validInput,
|
|
455
|
-
invalidInput = 'foo',
|
|
456
|
-
expectedTags,
|
|
457
|
-
ruleNames,
|
|
458
|
-
onFailureExpectedTags = [],
|
|
459
|
-
}) => {
|
|
460
|
-
const methodNames = ruleNames.map((n) => `${n}()`).join('.');
|
|
461
|
-
const createTagsObj = (str, tags) => tags.reduce((acc, curr) => {
|
|
462
|
-
acc[curr] = [0, str.length - 1];
|
|
463
|
-
|
|
464
|
-
return acc;
|
|
465
|
-
}, {});
|
|
466
|
-
|
|
467
|
-
it(`tracked values passing validation for ${methodNames} chain will be tagged with ${expectedTags.join(', ')}`, function() {
|
|
468
|
-
simulateRequestScope(() => {
|
|
469
|
-
const tracked = trackString(validInput);
|
|
470
|
-
let schema = joi.string();
|
|
471
|
-
|
|
472
|
-
for (const name of ruleNames) {
|
|
473
|
-
schema = schema[name]();
|
|
474
|
-
}
|
|
475
|
-
|
|
476
|
-
const validated = schema.validate(tracked, { convert: false });
|
|
477
|
-
const data = tracker.getData(validated.value);
|
|
478
|
-
|
|
479
|
-
expect(validInput).to.equal(validated.value);
|
|
480
|
-
|
|
481
|
-
expect(data.tags).to.deep.equal({
|
|
482
|
-
[DataflowTag.UNTRUSTED]: [0, validated.value.length - 1],
|
|
483
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, validated.value.length - 1],
|
|
484
|
-
...createTagsObj(validated.value, expectedTags)
|
|
485
|
-
});
|
|
486
|
-
});
|
|
487
|
-
});
|
|
488
|
-
|
|
489
|
-
it(`tracked values in ${methodNames} should not be tagged with failed chain validation`, function() {
|
|
490
|
-
simulateRequestScope(() => {
|
|
491
|
-
const tracked = trackString(invalidInput);
|
|
492
|
-
let schema = joi.string();
|
|
493
|
-
|
|
494
|
-
for (const name of ruleNames) {
|
|
495
|
-
schema = schema[name]();
|
|
496
|
-
}
|
|
497
|
-
|
|
498
|
-
const validated = schema.validate(tracked);
|
|
499
|
-
const data = tracker.getData(validated.value);
|
|
500
|
-
|
|
501
|
-
expect(data.tags).to.deep.equal({
|
|
502
|
-
[DataflowTag.UNTRUSTED]: [0, validated.value.length - 1],
|
|
503
|
-
[DataflowTag.STRING_TYPE_CHECKED]: [0, validated.value.length - 1],
|
|
504
|
-
...createTagsObj(validated.value, onFailureExpectedTags)
|
|
505
|
-
});
|
|
506
|
-
});
|
|
507
|
-
});
|
|
508
|
-
}
|
|
509
|
-
);
|
|
510
|
-
});
|
|
511
|
-
});
|
|
512
|
-
|
|
513
|
-
|
|
@@ -1,42 +0,0 @@
|
|
|
1
|
-
'use strict';
|
|
2
|
-
|
|
3
|
-
const sinon = require('sinon');
|
|
4
|
-
const { expect } = require('chai');
|
|
5
|
-
const { initAssessFixture } = require('@contrast/test/fixtures');
|
|
6
|
-
|
|
7
|
-
describe('assess dataflow propagation mongoose', function () {
|
|
8
|
-
let core, instr;
|
|
9
|
-
|
|
10
|
-
beforeEach(function () {
|
|
11
|
-
({ core } = initAssessFixture());
|
|
12
|
-
instr = core.assess.dataflow.propagation.mongooseInstrumentation;
|
|
13
|
-
});
|
|
14
|
-
|
|
15
|
-
const propagatorList = [
|
|
16
|
-
'schemaMap',
|
|
17
|
-
'schemaMixed',
|
|
18
|
-
'schemaString'
|
|
19
|
-
];
|
|
20
|
-
|
|
21
|
-
it('composes different installers', function () {
|
|
22
|
-
propagatorList.forEach((name) => {
|
|
23
|
-
expect(instr[name]).to.have.property('install').and.be.a('function');
|
|
24
|
-
});
|
|
25
|
-
});
|
|
26
|
-
|
|
27
|
-
it('dispatches installation to sub-components', function () {
|
|
28
|
-
const installs = [];
|
|
29
|
-
propagatorList.forEach((name) => {
|
|
30
|
-
const instr = core.assess.dataflow.propagation.mongooseInstrumentation;
|
|
31
|
-
installs.push(sinon.stub(instr[name], 'install'));
|
|
32
|
-
});
|
|
33
|
-
|
|
34
|
-
instr.install();
|
|
35
|
-
|
|
36
|
-
expect(installs).to.have.lengthOf(propagatorList.length);
|
|
37
|
-
installs.forEach((installer) => {
|
|
38
|
-
expect(installer).to.have.been.called;
|
|
39
|
-
});
|
|
40
|
-
});
|
|
41
|
-
});
|
|
42
|
-
|