-
Notifications
You must be signed in to change notification settings - Fork 335
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
chore: Update ic-agent and ic-agent's reverse dependencies to v0.39 #3282
Conversation
1b2ffd0
to
1289183
Compare
c04cbfc
to
7fe8cbc
Compare
db85b04
to
ec17eb9
Compare
984b7af
to
029c50c
Compare
@nikolay-komarevskiy : could you take a look at the failing test? I wasn't able to figure out what's breaking it :( |
@anchpop Thanks a lot for doing a very valuable effort of updating this dependency! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Rosetta-related changes LGTM, thanks @anchpop !
db9ed03
to
febda61
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for doing this!
…3282) I created this PR by following these instructions from @nathanosdev: > The following crates should use version 3.0.2: > 1. `ic-response-verification` > 2. `ic-certification` > 3. `ic-certificate-verification` > 4. `ic-http-certification` > 5. `ic-cbor` > > I recommend just setting them to `3` in `Cargo.toml` and `^3.0.2` in `external_crates.bzl` so they can be updated more easily in the future. That's how it's setup in `ic-http-gateway` too. > And `ic-http-gateway` can be set to version `0.1.0`. The git tag reference can be removed now. I also updated `ic-agent` to 0.39 because that is the version of `ic-agent` that `ic-http-gateway` v0.1.0 depends on. In doing so, I also updated `ic-utils` and `ic-transport-types`. I also moved the `ic-http-gateway` dependency to the workspace-level `Cargo.toml`. It seems like there were a number of breaking changes in these crates, so I also went through and fixed all the code that became broken. --------- Co-authored-by: IDX GitLab Automation <idx@dfinity.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Left a comment about not checking the cert in Agent.call
.
I created this PR by following these instructions from @nathanosdev:
I also updated
ic-agent
to 0.39 because that is the version ofic-agent
thatic-http-gateway
v0.1.0 depends on. In doing so, I also updatedic-utils
andic-transport-types
. I also moved theic-http-gateway
dependency to the workspace-levelCargo.toml
.It seems like there were a number of breaking changes in these crates, so I also went through and fixed all the code that became broken.