| パッケージ | 影響を受けるバージョン | 修正済みバージョン |
|---|---|---|
| @deepstream/server | ≥ 10.1.0, < 10.1.1 | 10.1.1 |
The RECORD_ACTION.PATCH_MULTI action is not registered in the Valve permission system's RULES_MAP (src/services/permission/valve/rules-map.ts). When ConfigPermission.canPerformAction() is called for a PATCH_MULTI message, getRulesForMessage() returns null because the action is missing from the map. This triggers an unconditional allow (callback(..., null, true)), completely bypassing all configured Valve permission rules.
Any authenticated user — regardless of their configured permissions — can write arbitrary data to any record using the PATCH_MULTI action.
In src/services/permission/valve/rules-map.ts lines 38-54, the RULES_MAP[TOPIC.RECORD].actions dictionary maps record actions to permission rule types. The actions registered include: SUBSCRIBE, SUBSCRIBEANDHEAD, SUBSCRIBEANDREAD, READ, HEAD, LISTEN, CREATE, UPDATE, PATCH, NOTIFY, DELETE, ERASE. However, RECORD_ACTION.PATCH_MULTI is absent from this map.
When getRulesForMessage() at line 86-99 encounters an action not in the map, it returns null. In config-permission.ts at line 88-93, when ruleSpecification === null, the callback is invoked with true (allow) unconditionally.
{topic: RECORD, action: PATCH_MULTI, name: "admin/secret-record", parsedData: [{path: "role", data: "admin"}]}message-processor.ts:68 invokes permission checkconfig-permission.ts:89 → getRulesForMessage() returns null for PATCH_MULTIconfig-permission.ts:92 → unconditional ALLOWpermission.type is set to config (Valve) — the recommended production configuration per deepstream documentationこれらの製品をお使いですか？サイトをスキャンして、脆弱なバージョンが公開されていないか確認しましょう。
無料でサイトをスキャンnone (OpenPermission) allows everything already, so default deployments are unaffected// Connect as a minimal-privilege user
const { DeepstreamClient } = require('@deepstream/client');
const client = new DeepstreamClient('localhost:6020');
await client.login({ username: 'restricted-user', password: 'password' });
// This should be blocked by Valve permissions but isn't:
// Send raw PATCH_MULTI message to bypass all permission rules
const connection = client.getConnection();
connection.sendMessage({
topic: 0x52, // TOPIC.RECORD
action: 0x50, // RECORD_ACTION.PATCH_MULTI (check actual enum value)
name: 'admin/protected-record',
parsedData: [
{ path: 'permissions', data: 'admin' },
{ path: 'secret', data: 'overwritten' }
]
});
Add PATCH_MULTI to the RULES_MAP in src/services/permission/valve/rules-map.ts:
[RECORD_ACTION.PATCH_MULTI]: RULE_TYPES.WRITE,
This maps PATCH_MULTI operations to the same WRITE permission rule that governs UPDATE and PATCH.
All versions that include PATCH_MULTI support with the Valve (ConfigPermission) permission system. The PATCH_MULTI action was added in commit 82ffa8119d8f4a8242ac5c3507469a22de746b65 but was never registered in RULES_MAP.
Vulnerability discovered by Zhixi "Jace" Sun of ASM/VI at TikTok.