|
delete params.query.$client; |
While whitelisting is a good security feature, it is the adapter's job to complain about invalid query parameters, not ours.
It makes it difficult to compose hooks with this as it damages the context if used at app level/around/provider and also in services.
Proposal:
const foreignParamCount = 0
// in an Object.keys(ctx.params.query?.$client) loop, foreignParamCount++
if (foreignParamCount === 0) {
delete ctx.params.query.$client
}
The alternative is to dump repetitive code like this in all over userland hooks:
delete ctx.params.query.$client.sudo
if (Object.keys(ctx.params.query.$client).length === 0) {
delete ctx.params.query.$client
}
feathers-hooks-common/src/hooks/params-from-client.ts
Line 22 in 15ddaa6
While whitelisting is a good security feature, it is the adapter's job to complain about invalid query parameters, not ours.
It makes it difficult to compose hooks with this as it damages the context if used at app level/around/provider and also in services.
Proposal:
The alternative is to dump repetitive code like this in all over userland hooks: