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

Fixes #1964 to catch general exception in z_sendmany and catch exceptions as reference-to-const. #1976

Merged
merged 1 commit into from
Jan 18, 2017

Conversation

bitcartel
Copy link
Contributor

No description provided.

@bitcartel bitcartel added A-rpc-interface Area: RPC interface A-wallet Area: Wallet labels Jan 3, 2017
@bitcartel bitcartel added this to the 1.0.5 milestone Jan 3, 2017
@bitcartel bitcartel self-assigned this Jan 3, 2017
Copy link
Contributor

@str4d str4d left a comment

Choose a reason for hiding this comment

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

utACK

@bitcartel bitcartel added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 17, 2017
@ebfull
Copy link
Contributor

ebfull commented Jan 18, 2017

#1964 (cross linking)

@ebfull
Copy link
Contributor

ebfull commented Jan 18, 2017

@zkbot r+

@zkbot
Copy link
Contributor

zkbot commented Jan 18, 2017

📌 Commit fce7260 has been approved by ebfull

@zkbot
Copy link
Contributor

zkbot commented Jan 18, 2017

⌛ Testing commit fce7260 with merge 1e8778a...

zkbot added a commit that referenced this pull request Jan 18, 2017
…=ebfull

Fixes #1964 to catch general exception in z_sendmany and catch exceptions as reference-to-const.
@zkbot
Copy link
Contributor

zkbot commented Jan 18, 2017

☀️ Test successful - zcash

@zkbot zkbot merged commit fce7260 into zcash:master Jan 18, 2017
ondrejsika added a commit to ondrejsika/zcash that referenced this pull request Jan 25, 2017
Zcash 1.0.5

This release contains a number of bug fixes and minor usability improvements, including:

1. The chain is now fully rescanned when keys are imported that are older than the wallet. (zcash#1978)
2. The number of commitments in the note commitment tree is now displayed by `getblockchaininfo`. (zcash#1946)
3. `zcash.conf` now must exist in order to start zcashd. (zcash#2013)
4. Fixed a bug where `z_sendmany` logged incorrect txid fragments when sending from transparent addresses. (zcash#1980)
5. We integrated upstream's cookie-based RPC authentication. (zcash#1999)
6. We added a restriction to wallet export paths to protect user security. (zcash#2006)
7. `z_getoperationstatus` is now sorted chronologically. (zcash#2015)
8. Messages containing newlines are now rendered properly by the metrics UI. (zcash#1972)
9. We added more tools for benchmarking JoinSplit creation. (zcash#1953)
10. We now show serialized transaction size in `listtransactions`, more operation details in `z_getoperationstatus`, and the age of the note being spent in `z_sendmany` logging. (zcash#2001, zcash#1976, zcash#1977)
11. We now instruct users to run `fetch-params` if the parameters could not be found locally. (zcash#1979)
12. We handle exceptions better in some situations for more user-friendly error messages. (zcash#1976)

For a more complete list of changes, see our [1.0.5 milestone](https://github.com/zcash/zcash/milestone/49?closed=1).
jmprcx added a commit to z-classic/zclassic that referenced this pull request Jan 31, 2017
Zcash 1.0.5

This release contains a number of bug fixes and minor usability improvements, including:

1. The chain is now fully rescanned when keys are imported that are older than the wallet. (zcash#1978)
2. The number of commitments in the note commitment tree is now displayed by `getblockchaininfo`. (zcash#1946)
3. `zcash.conf` now must exist in order to start zcashd. (zcash#2013)
4. Fixed a bug where `z_sendmany` logged incorrect txid fragments when sending from transparent addresses. (zcash#1980)
5. We integrated upstream's cookie-based RPC authentication. (zcash#1999)
6. We added a restriction to wallet export paths to protect user security. (zcash#2006)
7. `z_getoperationstatus` is now sorted chronologically. (zcash#2015)
8. Messages containing newlines are now rendered properly by the metrics UI. (zcash#1972)
9. We added more tools for benchmarking JoinSplit creation. (zcash#1953)
10. We now show serialized transaction size in `listtransactions`, more operation details in `z_getoperationstatus`, and the age of the note being spent in `z_sendmany` logging. (zcash#2001, zcash#1976, zcash#1977)
11. We now instruct users to run `fetch-params` if the parameters could not be found locally. (zcash#1979)
12. We handle exceptions better in some situations for more user-friendly error messages. (zcash#1976)

For a more complete list of changes, see our [1.0.5 milestone](https://github.com/zcash/zcash/milestone/49?closed=1).

Conflicts:
	README.md
	configure.ac
	contrib/debian/changelog
	contrib/debian/control
	contrib/debian/manpages/zcash-cli.1
	contrib/debian/manpages/zcashd.1
	contrib/gitian-descriptors/gitian-linux.yml
	doc/authors.md
	src/Makefile.gtest.include
	src/chainparams.cpp
	src/clientversion.h
	src/init.cpp
	src/main.h
	src/metrics.cpp
	src/metrics.h
	src/util.cpp
	src/wallet/asyncrpcoperation_sendmany.cpp
	src/wallet/asyncrpcoperation_sendmany.h
	src/wallet/rpcwallet.cpp
	src/zcbenchmarks.cpp
	zcutil/build-debian-package.sh
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rpc-interface Area: RPC interface A-wallet Area: Wallet S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants