Skip to content

feat: add check-endpoints command #85

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

lpahlavi
Copy link
Contributor

@lpahlavi lpahlavi commented Aug 4, 2025

(XC-391) Add the check-endpoints command which allows comparing the endpoints exposed in a canister's WASM against those exposed in the canister's Candid interface file (i.e. .did file) to ensure no unexpected endpoints are exposed. The command is only added to the ic-wasm binary if the check-endpoints feature is enabled.

The check-endpoints command also has a --hidden flag to allow endpoints being explicitly hidden from the Candid interface (e.g. an http_request endpoint).

A follow-up PR will enable using the new command within a new ic-wasm GitHub action to verify a canister's endpoints against its Candid interface within a CI pipeline.

@lpahlavi lpahlavi force-pushed the lpahlavi/XC-391-add-check-endpoints-command branch from c7e7916 to 6f13643 Compare August 5, 2025 09:39
@lpahlavi lpahlavi changed the base branch from main to lpahlavi/fix-info-without-serde August 5, 2025 09:43
@lpahlavi lpahlavi force-pushed the lpahlavi/XC-391-add-check-endpoints-command branch from 672d3fb to fdb03c0 Compare August 5, 2025 13:22
@lpahlavi lpahlavi force-pushed the lpahlavi/XC-391-add-check-endpoints-command branch from d11dc0a to a127e6c Compare August 5, 2025 13:57
Base automatically changed from lpahlavi/fix-info-without-serde to main August 7, 2025 15:13
@lpahlavi lpahlavi marked this pull request as ready for review August 8, 2025 07:58
@lpahlavi lpahlavi requested a review from a team as a code owner August 8, 2025 07:58
@lpahlavi lpahlavi requested a review from lwshang August 8, 2025 07:59
@gregorydemay gregorydemay self-requested a review August 11, 2025 08:52
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.

1 participant