Skip to content

feat: display proxy canister principal in network status#357

Merged
lwshang merged 4 commits intomainfrom
lwshang/SDK-2378
Feb 12, 2026
Merged

feat: display proxy canister principal in network status#357
lwshang merged 4 commits intomainfrom
lwshang/SDK-2378

Conversation

@lwshang
Copy link
Contributor

@lwshang lwshang commented Feb 12, 2026

The canister_call_through_proxy test is simplified.

lwshang and others added 4 commits February 12, 2026 12:43
Extends the `icp network status` command to display the proxy canister
principal alongside the existing Candid UI principal field. The field
appears in both text and JSON output formats when a proxy canister is
deployed on the managed network.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
The proxy canister is now automatically installed when starting a
managed network, so the test no longer needs to manually deploy it.
Instead, the test retrieves the proxy canister ID from
`icp network status --json` using the proxy_canister_principal field.

This simplifies the test and demonstrates the real-world usage pattern.
Also removes the vendored proxy.wasm file as it's no longer needed.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Documents the addition of proxy_canister_principal field to the
`icp network status` command output.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@lwshang lwshang marked this pull request as ready for review February 12, 2026 18:01
@lwshang lwshang requested a review from a team as a code owner February 12, 2026 18:01
@lwshang lwshang enabled auto-merge (squash) February 12, 2026 18:04
@lwshang lwshang merged commit 67beabd into main Feb 12, 2026
78 checks passed
@lwshang lwshang deleted the lwshang/SDK-2378 branch February 12, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants