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

Fix listreceivedbyaddress not taking address as a string #14417

Merged

Conversation

etscrivner
Copy link
Contributor

Fixes #14173. Add the patch in #14173 and include a regression test.

@etscrivner etscrivner force-pushed the fix-rpc-calls-not-taking-valid-strings branch from e57140b to ccdcd9e Compare October 6, 2018 17:14
@kristapsk
Copy link
Contributor

ACK ccdcd9eec9825cc95a75816637960511d67305c1

@meshcollider
Copy link
Contributor

utACK ccdcd9e

Note that your commit isn't associated with your GitHub account currently, did you use the same email address for both?

@etscrivner etscrivner force-pushed the fix-rpc-calls-not-taking-valid-strings branch from ccdcd9e to f80c9ef Compare October 14, 2018 16:43
Fixes bitcoin#14173. Add the patch in bitcoin#14173 and include a regression test.
@etscrivner etscrivner force-pushed the fix-rpc-calls-not-taking-valid-strings branch from f80c9ef to d4d70ed Compare October 14, 2018 16:48
@etscrivner
Copy link
Contributor Author

@meshcollider Thanks! The commit should have the correct email now.

@maflcko
Copy link
Member

maflcko commented Oct 23, 2018

utACK d4d70ed

@maflcko maflcko merged commit d4d70ed into bitcoin:master Oct 23, 2018
maflcko pushed a commit that referenced this pull request Oct 23, 2018
d4d70ed Fix listreceivedbyaddress not taking address as a string (Eric Scrivner)

Pull request description:

  Fixes #14173. Add the patch in #14173 and include a regression test.

Tree-SHA512: 5a9794e0c43e90d18c899841afbaf15eb9129d7d2f6570fccf0a1793697fe170d224c3c3995b1a35c536fac19819042823d9e3bd23b019d0f03434499243d2f5
@etscrivner etscrivner deleted the fix-rpc-calls-not-taking-valid-strings branch October 23, 2018 23:08
@promag
Copy link
Contributor

promag commented Nov 12, 2018

utACK d4d70ed.

@gmaxwell
Copy link
Contributor

Backport me?

maflcko pushed a commit to maflcko/bitcoin-core that referenced this pull request Nov 28, 2018
Fixes bitcoin#14173. Add the patch in bitcoin#14173 and include a regression test.

Github-Pull: bitcoin#14417
Rebased-From: d4d70ed
@maflcko maflcko added this to the 0.17.1 milestone Nov 28, 2018
deadalnix pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request May 22, 2020
Summary:
d4d70eda33 Fix listreceivedbyaddress not taking address as a string (Eric Scrivner)

Pull request description:

  Fixes #14173. Add the patch in #14173 and include a regression test.

Tree-SHA512: 5a9794e0c43e90d18c899841afbaf15eb9129d7d2f6570fccf0a1793697fe170d224c3c3995b1a35c536fac19819042823d9e3bd23b019d0f03434499243d2f5

Backport of Core [[bitcoin/bitcoin#14417 | PR14417]]

Test Plan:
  ninja
  ninja check
  ninja check-functional
Verify the changes to `wallet_listreceivedby` causes it to fail on master if the changes to `rpc/client.cpp` from this patch are reverted.

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D6232
Munkybooty pushed a commit to Munkybooty/dash that referenced this pull request Jul 21, 2021
…a string

d4d70ed Fix listreceivedbyaddress not taking address as a string (Eric Scrivner)

Pull request description:

  Fixes bitcoin#14173. Add the patch in bitcoin#14173 and include a regression test.

Tree-SHA512: 5a9794e0c43e90d18c899841afbaf15eb9129d7d2f6570fccf0a1793697fe170d224c3c3995b1a35c536fac19819042823d9e3bd23b019d0f03434499243d2f5
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Aug 16, 2021
…a string

d4d70ed Fix listreceivedbyaddress not taking address as a string (Eric Scrivner)

Pull request description:

  Fixes bitcoin#14173. Add the patch in bitcoin#14173 and include a regression test.

Tree-SHA512: 5a9794e0c43e90d18c899841afbaf15eb9129d7d2f6570fccf0a1793697fe170d224c3c3995b1a35c536fac19819042823d9e3bd23b019d0f03434499243d2f5
vijaydasmp pushed a commit to vijaydasmp/dash that referenced this pull request Sep 8, 2021
…a string

d4d70ed Fix listreceivedbyaddress not taking address as a string (Eric Scrivner)

Pull request description:

  Fixes bitcoin#14173. Add the patch in bitcoin#14173 and include a regression test.

Tree-SHA512: 5a9794e0c43e90d18c899841afbaf15eb9129d7d2f6570fccf0a1793697fe170d224c3c3995b1a35c536fac19819042823d9e3bd23b019d0f03434499243d2f5
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rpc method listreceivedbyaddress doesn't take address as a string
8 participants