Skip to content

Commit cd4065c

Browse files
sirtimidclaude
andcommitted
fix(evm-wallet-experiment): docker e2e ALLOWED_HOSTS must be hostnames only
After PR #942 (Snaps network endowment factory) introduced the `network-caveat.ts` matcher with strict "hostname-only, no port" semantics (see packages/ocap-kernel/src/vats/network-caveat.ts:17-20), the e2e allowlist entries `['evm:8545', 'bundler:4337']` never match `URL.hostname` (which is just `'evm'` / `'bundler'`). Every fetch out of the provider vat was rejected with "Invalid host: evm", causing every docker e2e test to fail at `createSmartAccount`. Drop the ports to match the caveat's contract. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent eef0379 commit cd4065c

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

  • packages/evm-wallet-experiment/test/e2e/docker/helpers

packages/evm-wallet-experiment/test/e2e/docker/helpers/scenarios.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,10 @@ const TEST_MNEMONIC =
3232
const CHAIN_ID = 31337;
3333
const EVM_RPC_URL = 'http://evm:8545';
3434
const BUNDLER_URL = 'http://bundler:4337';
35-
const ALLOWED_HOSTS = ['evm:8545', 'bundler:4337'];
35+
// Hostnames only — the kernel's network caveat (packages/ocap-kernel
36+
// /src/vats/network-caveat.ts) matches `URL.hostname`, which never
37+
// includes a port.
38+
const ALLOWED_HOSTS = ['evm', 'bundler'];
3639

3740
export type HomeResult = {
3841
kref: string;

0 commit comments

Comments
 (0)