Skip to content

Commit 0668a04

Browse files
committed
fix: preserve socket data in API responses
1 parent a092f17 commit 0668a04

2 files changed

Lines changed: 2 additions & 3 deletions

File tree

‎src/server.js‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ let crud = null;
3333

3434
function sendToServer(data) {
3535
const envelope = { ...data };
36-
delete envelope.socket;
3736
return server.send(envelope);
3837
}
3938

‎test/send-routing.test.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ test('API responses and errors enter server.send instead of raw socket delivery'
1414
assert.equal(await send(data), data);
1515
assert.deepEqual(data.data, { ok: true });
1616
assert.notEqual(sent[0], data);
17-
assert.equal(sent[0].socket, undefined);
17+
assert.equal(sent[0].socket, data.socket);
1818
assert.equal(sent[0].socketId, 'trusted-socket');
1919
assert.equal(sent[0].clientId, 'trusted-client');
2020
const missing = { organization_id: 'tenant', host: 'tenant.test' };
@@ -41,7 +41,7 @@ test('API AST socket operators retain tenant scope while entering server.send',
4141
assert.equal(sent[0].organization_id, 'tenant');
4242
assert.equal(sent[0].host, 'tenant.test');
4343
assert.equal(sent[0].socket, undefined);
44-
assert.equal(sent[1].socket, undefined);
44+
assert.equal(sent[1].socket, data.socket);
4545
assert.equal(sent[1].socketId, 'trusted-socket');
4646
assert.equal(sent.length, 2);
4747
});

0 commit comments

Comments
 (0)