Require zcash.conf to start zcashd#2013
Merged
zkbot merged 2 commits intozcash:masterfrom Jan 16, 2017
Merged
Conversation
An empty zcash.conf is sufficient to bypass this error.
Contributor
|
ACK |
Contributor
|
ACK @zkbot r+ |
Contributor
|
📌 Commit 24f4e33 has been approved by |
Contributor
|
⌛ Testing commit 24f4e33 with merge 5738aaa... |
Contributor
|
💔 Test failed - zcash |
Contributor
|
Wallet tests in gtest are failing, so... this restriction on the zcash.conf must be interfering with tests or test initialization? |
Contributor
Author
|
The tests that failed pass locally for me. So there's an environmental effect. |
Contributor
Author
|
Confirmed: it's a regression in an earlier PR. This PR and the next one failed in the same way, because the build worker hasn't been idle long enough to be shut down and wiped. |
Contributor
Author
|
@zkbot retry |
Contributor
zkbot
added a commit
that referenced
this pull request
Jan 16, 2017
Require zcash.conf to start zcashd Closes #2002.
Contributor
|
☀️ Test successful - zcash |
zkbot
added a commit
that referenced
this pull request
Jan 17, 2017
Create empty zcash.conf during performance measurements Fixes benchmarking regression caused by #2013.
str4d
added a commit
to str4d/zcash
that referenced
this pull request
Jan 18, 2017
Fixes regression caused by zcash#2013.
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
jmprcx
added a commit
to z-classic/zclassic
that referenced
this pull request
Jan 31, 2017
* Add getlocalsolps and getnetworksolps RPC calls, show them in getmininginfo * Add benchmark for attempting decryption of notes * Add benchmark for incrementing note witnesses * Add -metricsui flag to toggle between persistent screen and rolling metrics Defaults to true if stdout is a TTY, else false. * Add -metricsrefreshtime option * Only show metrics by default if stdout is a TTY * Document metrics screen options * Fix stale comment referencing upstream block interval * Add checkpoint at block height 15000 * Make command line option to show all debugging consistent with similar options Most people expect a value of 1 to enable all for command line arguments. However to do this for the -debug option you must type "-debug=". This has been changed to allow "-debug=1" as well as "-debug=" to enable all debug logging * Update documentation to match the zcash#4219 change * Update help message to match the zcash#4219 change * Clarify that metrics options are only useful without -daemon and -printtoconsole * Increase length of metrics divider * Closes zcash#1857. Fixes bug where tx spending only notes had priority of 0. * Closes zcash#1901. Increase default settings for the max block size when mining and the amount of space available for priority transactions. * Write witness caches when writing the best block For steady-state operation, this reduces the average time between wallet disk writes from once per block to once per hour. On -rescan, witness caches are only written out at the end along with the best block, increasing speed while ensuring that on-disk state is kept consistent. Witness caches are now never recreated during a -reindex, on the assumption that the blocks themselves are not changing (the chain is just being reconstructed), and so the witnesses will remain valid. Part of zcash#1749. * Add porter dev overrides for CC, CXX, MAKE, BUILD, HOST * Apply miniupnpc patches to enable compilation on Solaris 11 These can be removed after the next MiniUPnP release. Closes zcash#1835. * Closes zcash#1903. Add fee parameter to z_sendmany. * Add an upstream miniupnpc patch revision * Metrics - Don't exclaim unless > 1 "You have validated 0 transactions!" sounds a little less enthusiastic that intended. Also, only says "1 transaction". * Address review comments, tweak strings * bash-completion: Adapt for 0.12 and 0.13 * separate completion for bitcoind and bitcoin-cli * remove RPC support from bitcoind completion * add completion for bitcoin-tx and bitcoin-qt * rely on autoloading of completions * Change function names to not clash with Bitcoin, apply to correct binaries * Add bash completion files to Debian package * Always bash-complete the default account * Add Zcash RPC commands to CLI argument completion * Fixes zcash#1823. Witness anchors for input notes no longer cross block boundaries. * Edit for grammar: "block chain" At this point, I believe it is universally accepted that "blockchain" is one word, and should not be separated into two. * Increase timeout as laptops on battery power have cpu throttling. * Isolate verification to a `ProofVerifier` context object that allows verification behavior to be tuned by the caller. * Regression test. * Ensure cache contains valid entry when anchor is popped. * Ensure ProofVerifier cannot be accidentally copied. * Document behaviour of CWallet::SetBestChain * WitnessAnchorData only needs to store one witness per JSOutPoint. * Rename Dummy to Disabled. * Add more tests for ProofVerifier. * Fix indentation * Generate JS for trydecryptnotes, make number of addresses a variable * Add JS to second block to ensure witnesses are incremented * ASSERT_TRUE -> ASSERT_FALSE * Skip JoinSplit verification before the last checkpoint Part of zcash#1749 * Gather release notes from previous release to HEAD Also update release-process.md to replace git shortlog command with release-notes.py script. * Add a reindex test that fails because of a bug in decrementing witness caches Ref: zcash#1904 (comment) * Make the test pass by fixing the bug! * Only check cache validity for witnesses being incremented or decremented Fixes the bug resulting from zcash#1904. * Update release process to check in with users who opened resolved issues * Check that E' points are actually in G2 by ensuring they are of order r. * Fix bug in wallet tests * Extract block-generation wallet test code into a function * Rewrite reindex test to check beyond the max witness cache size * Fix bug in IncrementNoteWitness() * Extend createjoinsplit to benchmark parallel JoinSplits Closes zcash#1940 * Update payment API docs to recommend -rescan for fixing witness errors * Add total number of commitments to getblockchaininfo * Update version to 1.0.4 * Update man pages * Release notes, authors, changelog * Only enable getblocktemplate when wallet is enabled * Only run wallet tests when wallet is enabled * Add a tool for profiling the creation of JoinSplits * Add test for IncrementalMerkleTree::size(). * Exclude test binaries from make install Closes zcash#1943. * Fixes zcash#1964 to catch general exception in z_sendmany and catch exceptions as reference-to-const. * Fixes zcash#1967 by adding age of note to z_sendmany logging. * Scan the whole chain whenever a z-key is imported Closes zcash#1941. * Instruct users to run zcash-fetch-params if network params aren't available Closes zcash#1786. * Fixes a bug where the unsigned transaction was logged by z_sendmany after a successful sign and send, meaning that the logged hash fragment would be different from the txid logged by "AddToWallet". This issue occured when sending from transparent addresses, as utxo inputs must be signed. It did not occur when sending from shielded addresses. * Trigger metrics UI refresh on new messages * Strip out the SECURE flag in metrics UI so message style is detected * Add 'CreateJoinSplit' standalone utility to gitignore. * Handle newlines in UI messages * Suggest ./zcutil/fetch-params.sh as well Once we improve the from-source installation docs to use 'make install', we can revert this commit. * Update debug categories Closes zcash#1954. * CreateJoinSplit: add start_profiling() call This solves the problem of profiling output displaying nonsensical large time values. * rpc: Implement random-cookie based authentication When no `-rpcpassword` is specified, use a special 'cookie' file for authentication. This file is generated with random content when the daemon starts, and deleted when it exits. Read access to this file controls who can access through RPC. By default this file is stored in the data directory but it be overriden with `-rpccookiefile`. This is similar to Tor CookieAuthentication: see https://www.torproject.org/docs/tor-manual.html.en Alternative to zcash#6258. Like that pull, this allows running bitcoind without any manual configuration. However, daemons should ideally never write to their configuration files, so I prefer this solution. * Rename build-aux/m4/bitcoin_find_bdb48.m4 to remove version Closes zcash#1622. * Bump COPYRIGHT_YEAR from 2016 to 2017. * Throw an error if zcash.conf is missing An empty zcash.conf is sufficient to bypass this error. * Show a friendly message explaining why zcashd needs a zcash.conf * Closes zcash#1780. Result of z_getoperationstatus now sorted by creation time of operation * Create ISSUE_TEMPLATE.md * move template to subdirectory, fix typo, include prompt under describing issue section, include uploading file directly to github ticket as option for sharing logs * Remove UTF-8 BOM efbbbf from zcash.conf to avoid problems with command line tools * Closes zcash#1097 so zcash-cli now displays license info like zcashd. LicenseInfo is refactored from init.cpp to util.cpp so that the bitcoin-cli makefile target does not need to be modified. * Fixes zcash#1497 ZCA-009 by restricting data exporting to user defined folder. Previously the RPC interface allowed z_exportwallet, backupwallet and dumpwallet to write data to an arbitrary filename. ZCA-009 demonstrates how this is vulnerable. The resolution is to only allow data to written when the -exportdir has been configured. Also filenames are restricted to alphanumeric characters. * Closes zcash#1957 by adding tx serialization size to listtransactions output. * Fix gtest ordering broken by zcash#1949 Part of zcash#1539 * Fixes zcash#1960: z_getoperationstatus/result now includes operation details. * Debian package lint - Tweak description synopsis to make Debian happy - Put bash completion files in correct directory - Add a manpage for zcash-fetch-params * Generate Debian control file to fix shlibs lint * Create empty zcash.conf during performance measurements * Create empty zcash.conf during coverage checks Fixes regression caused by zcash#2013. * Coverage build system tweaks * Update walletbackup.py qa test to use -exportdir option * Add missing header required by std::accumulate * Increase timeout for z_sendmany transaction in wallet.py qa test * Add test for z_importkey rescanning from beginning of chain. * Bump version to 1.0.5. * Update release notes and Debian package. * V1.0.4 mac (#51) * initial mac version of zclassic Work in progress - 15JAN2017 more refactoring linux refactoring fixes osx refactoring fixes initial win64 commit fixup! initial win64 commit compile libsnark with posix threads build gtest and gmock with posix Working build fixup! Working build * Windows and Linux builds ok * fixup! Merge tag 'v1.0.5' into v1.0.5-multios * fixup! fixup! Merge tag 'v1.0.5' into v1.0.5-multios * fixup! fixup! fixup! Merge tag 'v1.0.5' into v1.0.5-multios * Fix OSX compatibility with depends * OSX Compat - Fix site_t ambiguity in json * fixup! OSX Compat - Fix site_t ambiguity in json
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #2002.