Skip to content

Commit 53b3581

Browse files
committed
JS: Add test to stress flow through properties
1 parent 784991c commit 53b3581

File tree

4 files changed

+21
-0
lines changed

4 files changed

+21
-0
lines changed

javascript/ql/test/library-tests/Routing/route-setups.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -291,3 +291,10 @@ function withForLoop() {
291291
sink(req.taint); // NOT OK
292292
});
293293
}
294+
295+
function routeHandlersInProps() {
296+
let routes = require('./routes');
297+
const app = express();
298+
app.use(routes.first);
299+
app.get('/', routes.second);
300+
}
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
export default function first(req, res, next) {
2+
req.tainted = source();
3+
req.safe = 'safe';
4+
next();
5+
}
6+
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
export default {
2+
first: require('./first'),
3+
second: require('./second'),
4+
};
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
export default function second(req, res) {
2+
sink(req.tainted); // NOT OK
3+
sink(req.safe); // OK
4+
}

0 commit comments

Comments
 (0)