Skip to content
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

[release-v1.8] main: Use backported rpc types updates. #3192

Merged
merged 3 commits into from
Sep 27, 2023

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented Sep 27, 2023

This updates the 1.8 release to use the latest version of the rpc/jsonrpc/types module which includes updates to expose the new proof of work hash in the verbose getblock and getblockheader results.

In particular, the following updated module version is used:

Note that it also cherry picks all of the commits included in updates to the rpc/jsonrpc/types/v4 module to ensure they are also included in the release branch even though it is not strictly necessary since the go.mod has been updated to require the new rpc/jsonrpc/types/v4.1.0 release and thus will pull in the new code. However, from past experience, not having code backported to modules available in the release branch too leads to headaches for devs building from source in their local workspace with overrides such as those in go.work.

davecgh and others added 2 commits September 27, 2023 09:21
This updates the 1.8 release to use the latest version of the
rpc/jsonrpc/types module which includes updates to expose the new proof
of work hash in the verbose getblock and getblockheader results.

In particular, the following updated module version is used:

- github.com/decred/dcrd/rpc/jsonrpc/types/[email protected]
@davecgh davecgh added this to the 1.8.1 milestone Sep 27, 2023
@jrick
Copy link
Member

jrick commented Sep 27, 2023

This only changes the types, it doesn't backport the changes to the rpc method handlers that populate the fields?

Copy link
Member

@jrick jrick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's better :)

@davecgh
Copy link
Member Author

davecgh commented Sep 27, 2023

This only changes the types, it doesn't backport the changes to the rpc method handlers that populate the fields?

Aye, I noticed that and was in the process of updating. It should be good to go now.

@davecgh davecgh merged commit 1e06f6d into decred:release-v1.8 Sep 27, 2023
2 checks passed
@davecgh davecgh deleted the rel18_backports branch September 27, 2023 14:53
@davecgh davecgh added the rpc server api change Issues and/or pull requests that involve a new RPC server version or breaking to change to the API. label Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rpc server api change Issues and/or pull requests that involve a new RPC server version or breaking to change to the API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants