Skip to content

Conversation

lpahlavi
Copy link
Contributor

@lpahlavi lpahlavi commented Sep 30, 2025

(XC-502) Update the ic-cdk version to v0.18. This is a major upgrade of the ic-cdk and as such is a breaking change for canhttp.

BREAKING: There are multiple breaking changes in this PR:

  • IcError, IcHttpRequestWithCycles and http::CyclesCostEstimator are removed.
  • Client:
    • no longer implements Service<IcHttpRequestWithCycles>
    • now implements Service<ic_management_canister_types::HttpRequestArgs>
  • CyclesAccounting:
    • no longer implements Convert<ic_cdk::api::management_canister::http_request::CanisterHttpRequestArgument>
    • now implements Filter<ic_management_canister_types::HttpRequestArgs>
    • CyclesAccounting::new no longer takes a num_nodes_in_subnet argument.
  • CyclesChargingPolicy::charge_cycles now takes a ic_management_canister_types::HttpRequestArgs as argument.

@lpahlavi lpahlavi force-pushed the lpahlavi/XC-412-update-ic-cdk branch from 3ddd9de to 59aedd8 Compare October 2, 2025 06:49
@lpahlavi lpahlavi changed the title build: update ic-cdk to v0.18.7 build!: update ic-cdk to v0.18.7 Oct 2, 2025
@lpahlavi lpahlavi requested a review from gregorydemay October 2, 2025 07:27
@lpahlavi lpahlavi marked this pull request as ready for review October 2, 2025 07:27
@lpahlavi lpahlavi requested a review from a team as a code owner October 2, 2025 07:27
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