Add wallet backup text to import* and add* RPCs #11289

meshcollider · 2017-09-08T12:23:37Z

Adds "Requires a new wallet backup" text to addwitnessaddress, importprivkey, importmulti, importaddress, importpubkey, and addmultisigaddress. Also adds a warning to dumpwallet that backing up the seed alone is not sufficient to back up non-HD addresses

achow101

Concept ACK

achow101 · 2017-09-08T19:47:07Z

src/wallet/rpcdump.cpp

@@ -62,7 +62,7 @@ std::string DecodeDumpString(const std::string &str) {
    for (unsigned int pos = 0; pos < str.length(); pos++) {
        unsigned char c = str[pos];
        if (c == '%' && pos+2 < str.length()) {
-            c = (((str[pos+1]>>6)*9+((str[pos+1]-'0')&15)) << 4) | 
+            c = (((str[pos+1]>>6)*9+((str[pos+1]-'0')&15)) << 4) |


Unrelated whitespace change.

I mentioned that in the PR, is it an issue?

I isn't really a problem, but you should try to avoid making such changes. Atom can be set to not make any whitespace changes at all.

It is a problem, as it often breaks backporting cherry-picks.

achow101 · 2017-09-08T19:47:19Z

src/wallet/rpcdump.cpp

@@ -644,7 +644,7 @@ UniValue dumpwallet(const JSONRPCRequest& request)
    file << strprintf("#   mined on %s\n", EncodeDumpTime(chainActive.Tip()->GetBlockTime()));
    file << "\n";

-    // add the base58check encoded extended master if the wallet uses HD 
+    // add the base58check encoded extended master if the wallet uses HD


Unrelated whitespace change.

maflcko · 2017-09-09T21:16:41Z

Mind to add to dumpwallet that just backing up the private masterkey is likely insufficient?

meshcollider · 2017-09-09T23:09:31Z

Done 👍

meshcollider · 2017-09-11T06:24:12Z

Rebased to fix .bitcoin directory travis failure

achow101 · 2017-09-15T15:39:51Z

utACK 288dfaea9d987399b92514121f98a674e74d5580

TheBlueMatt

I suppose for completeness importwallet may need the (rather astoundingly obvious) "Requires a new backup" note.

I guess we dont want to note anything about keypool topup leading to needing backups in things like "getnewaddress" (for non-HD wallets)? What about explicit ones like "keypoolrefill"? And things that always wipe keypool like encryptwallet? I guess probably not?

Similarly, I suppose we dont care about listing things like move/sendfrom which change wallet fileds for F$#King accounts?

TheBlueMatt · 2017-09-21T21:18:24Z

src/wallet/rpcdump.cpp

@@ -601,6 +601,7 @@ UniValue dumpwallet(const JSONRPCRequest& request)
        throw std::runtime_error(
            "dumpwallet \"filename\"\n"
            "\nDumps all wallet keys in a human-readable format.\n"
+            "Note that if your wallet contains keys which are not derived from your HD seed (e.g. imported keys), these are not covered by only backing up the seed itself, and must be backed up too.\n"


This leaves me confused. The things I need to back up are in dumpwallet, just not the master key listed in dump wallet, right? So just the dumpwallet file is a sufficient backup (with rescan, including imported segwit addresses and p2sh redeemScripts?), but not only the master key?

I've changed it, lmk what you think now

mess110 · 2017-09-27T20:57:09Z

src/wallet/rpcdump.cpp

@@ -601,6 +601,7 @@ UniValue dumpwallet(const JSONRPCRequest& request)
        throw std::runtime_error(
            "dumpwallet \"filename\"\n"
            "\nDumps all wallet keys in a human-readable format.\n"


@TheBlueMatt would changing this string to something like Dumps all wallet derived keys in a human-readable format. help cleanup the confusion?

meshcollider · 2017-09-30T02:44:41Z

Rebased + hopefully clarified the text as per @TheBlueMatt comments

TheBlueMatt · 2017-10-02T19:57:02Z

Wait, now I'm even more confused. AFAICT, dumpwallet does dump all imported keys, not just "derived" keys, so no need to back those up. It does not, however, dump imported scripts, so witness addresses and multisig addresses will be missed, right?

meshcollider · 2017-10-02T21:52:52Z

That's what I meant by (e.g. ensure you back up the whole dumpfile) (note that I didn't follow the suggested edit by mess110 because it's wrong, is that what you were looking at?). Are you suggesting mentioning that those imported scripts aren't backed up, or?

TheBlueMatt · 2017-10-04T00:10:32Z

Hmm, maybe I missed the parenthetical. Do we want to mention anything about the segwit addresses and imported scripts?

meshcollider · 2017-10-04T05:34:28Z

The description does currently specifically refer to 'keys' not scripts or segwit addresses. But is there any reason scripts aren't also dumped too?

laanwj · 2017-10-05T10:25:13Z

The description does currently specifically refer to 'keys' not scripts or segwit addresses. But is there any reason scripts aren't also dumped too?

No. just that (AFAIK) no one implemented that yet. Ideally a dumpwallet would not lose any relevant information.

TheBlueMatt · 2017-10-05T21:31:43Z

I dont see why scripts shouldnt be dumped either. I'm not sure if its really in scope for this PR, but would be a welcome addition IMO.

meshcollider · 2017-10-05T21:43:02Z

@laanwj @TheBlueMatt I'll look into it and try get a PR open for that soon too then

meshcollider · 2017-10-13T22:49:21Z

Rebased

TheBlueMatt

utACK a38bfbc. Would like to see the dumpwallet-includes-imported-scripts followup.

TheBlueMatt · 2017-10-17T17:46:40Z

src/wallet/rpcdump.cpp

@@ -601,6 +601,7 @@ UniValue dumpwallet(const JSONRPCRequest& request)
        throw std::runtime_error(
            "dumpwallet \"filename\"\n"
            "\nDumps all wallet keys in a human-readable format to a server-side file. This does not allow overwriting existing files.\n"
+            "Note that if your wallet contains keys which are not derived from your HD seed (e.g. imported keys), these are not covered by only backing up the seed itself, and must be backed up too (e.g. ensure you back up the whole dumpfile).\n"


Note that you may want to wrap this line.

meshcollider · 2017-10-19T09:03:28Z

I've wrapped the text, and added a note about scripts not being included, until I get the other PR up. Will remove that note in the other PR when it happens.

JohnVonNeumann · 2017-11-07T23:42:48Z

src/wallet/rpcwallet.cpp

@@ -1110,7 +1110,7 @@ UniValue addmultisigaddress(const JSONRPCRequest& request)
    if (request.fHelp || request.params.size() < 2 || request.params.size() > 3)
    {
        std::string msg = "addmultisigaddress nrequired [\"key\",...] ( \"account\" )\n"
-            "\nAdd a nrequired-to-sign multisignature address to the wallet.\n"
+            "\nAdd a nrequired-to-sign multisignature address to the wallet. Requires a new wallet backup.\n"


"nrequired-to-sign multisignature"

Only comment, I'm assuming this isn't a typo, and is a reference to the number of required signatures needed for the multisig address?

Correct :) And note that that text is not changed in this PR, it was there before

Ok sweet, yeah I saw that it wasn't a change from this commit, but wasn't sure so figured I'd ask. Thanks! Is it typically frowned upon to comment on things not directly changed by the PR/Commit/Issue in question?

No, if something looks wrong then feel free to point it out :) I was just clarifying

Awesome, thanks again.

c098c58 Wrap dumpwallet warning and note scripts aren't dumped (MeshCollider) a38bfbc Add wallet backup text to import*, add* and dumpwallet RPCs (MeshCollider) Pull request description: Closes #11243 Adds "Requires a new wallet backup" text to `addwitnessaddress`, `importprivkey`, `importmulti`, `importaddress`, `importpubkey`, and `addmultisigaddress`. Also adds a warning to `dumpwallet` that backing up the seed alone is not sufficient to back up non-HD addresses Tree-SHA512: 76d7cdca54d5b458acf479154620322391b889922525fddd6153f4164cfee393ad743757400cb8f6b1b30f24947df68ea9043b4e509f7df77a8fa05dda370933

Github-Pull: bitcoin#11289 Rebased-From: a38bfbc

Github-Pull: bitcoin#11289 Rebased-From: c098c58

656fde5 Add script birthtime metadata to dump and import wallet (MeshCollider) 1bab9b2 Add script dump note to RPC help text and release notes (MeshCollider) 68c1e00 Add test for importwallet (MeshCollider) 9e1184d Add dumpwallet scripts test (MeshCollider) ef0c730 Add scripts to importwallet RPC (MeshCollider) b702ae8 Add CScripts to dumpwallet RPC (MeshCollider) cdc260a Add GetCScripts to CBasicKeyStore (MeshCollider) Pull request description: As discussed in #11289 (comment), adds the CScripts from the wallet to the `dumpwallet` RPC and then allows them to be imported with the `importwallet` RPC. Includes a basic test, and modifies the helptext of the dumpwallet RPC. Notes: - Reviewers: use `?w=1` to avoid the indentation-only change in commit `Add scripts to importwallet RPC ` - currently the scripts are followed with `# addr=` comments just as the other keys are, unsure if this might confuse users into thinking all the scripts are for valid P2SH addresses though, but I don't think that should be an issue. - there are no birthtimes for scripts, so script imports don't affect rescans - `importwallet` imports the CScripts but I'm not sure how to approach specifying whether scripts are for P2SH addresses, BIP173 addresses, etc. whether that matters or not. Otherwise the RPC helptext might just need modification. Fixes #11715 Tree-SHA512: 36c55837b3a58b9d3499d4c0c2ae82153d62aa71919e751574651b63a1d2b8ecc83796db4553cc65dad9b5341c3a42ae2fcf4d62598c30af267f8e1461ba8272

656fde5 Add script birthtime metadata to dump and import wallet (MeshCollider) 1bab9b2 Add script dump note to RPC help text and release notes (MeshCollider) 68c1e00 Add test for importwallet (MeshCollider) 9e1184d Add dumpwallet scripts test (MeshCollider) ef0c730 Add scripts to importwallet RPC (MeshCollider) b702ae8 Add CScripts to dumpwallet RPC (MeshCollider) cdc260a Add GetCScripts to CBasicKeyStore (MeshCollider) Pull request description: As discussed in bitcoin/bitcoin#11289 (comment), adds the CScripts from the wallet to the `dumpwallet` RPC and then allows them to be imported with the `importwallet` RPC. Includes a basic test, and modifies the helptext of the dumpwallet RPC. Notes: - Reviewers: use `?w=1` to avoid the indentation-only change in commit `Add scripts to importwallet RPC ` - currently the scripts are followed with `# addr=` comments just as the other keys are, unsure if this might confuse users into thinking all the scripts are for valid P2SH addresses though, but I don't think that should be an issue. - there are no birthtimes for scripts, so script imports don't affect rescans - `importwallet` imports the CScripts but I'm not sure how to approach specifying whether scripts are for P2SH addresses, BIP173 addresses, etc. whether that matters or not. Otherwise the RPC helptext might just need modification. Fixes #11715 Tree-SHA512: 36c55837b3a58b9d3499d4c0c2ae82153d62aa71919e751574651b63a1d2b8ecc83796db4553cc65dad9b5341c3a42ae2fcf4d62598c30af267f8e1461ba8272

c098c58 Wrap dumpwallet warning and note scripts aren't dumped (MeshCollider) a38bfbc Add wallet backup text to import*, add* and dumpwallet RPCs (MeshCollider) Pull request description: Closes bitcoin#11243 Adds "Requires a new wallet backup" text to `addwitnessaddress`, `importprivkey`, `importmulti`, `importaddress`, `importpubkey`, and `addmultisigaddress`. Also adds a warning to `dumpwallet` that backing up the seed alone is not sufficient to back up non-HD addresses Tree-SHA512: 76d7cdca54d5b458acf479154620322391b889922525fddd6153f4164cfee393ad743757400cb8f6b1b30f24947df68ea9043b4e509f7df77a8fa05dda370933

656fde5 Add script birthtime metadata to dump and import wallet (MeshCollider) 1bab9b2 Add script dump note to RPC help text and release notes (MeshCollider) 68c1e00 Add test for importwallet (MeshCollider) 9e1184d Add dumpwallet scripts test (MeshCollider) ef0c730 Add scripts to importwallet RPC (MeshCollider) b702ae8 Add CScripts to dumpwallet RPC (MeshCollider) cdc260a Add GetCScripts to CBasicKeyStore (MeshCollider) Pull request description: As discussed in bitcoin#11289 (comment), adds the CScripts from the wallet to the `dumpwallet` RPC and then allows them to be imported with the `importwallet` RPC. Includes a basic test, and modifies the helptext of the dumpwallet RPC. Notes: - Reviewers: use `?w=1` to avoid the indentation-only change in commit `Add scripts to importwallet RPC ` - currently the scripts are followed with `# addr=` comments just as the other keys are, unsure if this might confuse users into thinking all the scripts are for valid P2SH addresses though, but I don't think that should be an issue. - there are no birthtimes for scripts, so script imports don't affect rescans - `importwallet` imports the CScripts but I'm not sure how to approach specifying whether scripts are for P2SH addresses, BIP173 addresses, etc. whether that matters or not. Otherwise the RPC helptext might just need modification. Fixes bitcoin#11715 Tree-SHA512: 36c55837b3a58b9d3499d4c0c2ae82153d62aa71919e751574651b63a1d2b8ecc83796db4553cc65dad9b5341c3a42ae2fcf4d62598c30af267f8e1461ba8272

d5526bd Wrap dumpwallet warning and note scripts aren't dumped (MeshCollider) 3711c6a Add wallet backup text to import*, add* and dumpwallet RPCs (MeshCollider) dbda874 [Wallet] always show help-line of wallet encryption calls (Jonas Schnelli) 20c269b Avoid opening copied wallet databases simultaneously (Russell Yanofsky) e411b70 [wallet] Fix leak in CDB constructor (João Barbosa) f15aeea Change getmininginfo errors field to warnings (Andrew Chow) c04390b Unify help text for GetWarnings output in get*info RPCs (random-zebra) 1d966ce Add warnings field to getblockchaininfo (Andrew Chow) ffcd781 [Trivial] Cleanup after MOVE-ONLY commits (random-zebra) e067235 MOVEONLY: Init functions wallet/wallet.cpp -> wallet/init.cpp (random-zebra) e947eec MOVEONLY: Fee functions wallet/wallet.cpp -> wallet/fees.cpp (random-zebra) 2188c3e Move some static functions out of wallet.h/cpp (random-zebra) f49acf7 [wallet] [moveonly] Move CAffectedKeysVisitor (random-zebra) 8bd979f [wallet] Specify wallet name in wallet loading errors (random-zebra) 900bbfa Reject invalid wallet files (João Barbosa) a1f4e2a Reject duplicate wallet filenames (random-zebra) ee52c2e Fix misleading "Method not found" multiwallet errors (Russell Yanofsky) ce35e1e [Qt] Use wallet 0 in rpc console if running with multiple wallets (random-zebra) 37089d1 [tests] Update wallet_multiwallet.py functional test (random-zebra) 3955ee9 [Doc] Update release notes (random-zebra) 4fd5913 [wallet] [rpc] Add listwallets RPC (John Newbery) 1525281 [wallet] [rpc] print wallet name in getwalletinfo (John Newbery) fdf5da0 [wallet] fix comment for CWallet::Verify() (John Newbery) cf4a90b Remove factor of 3 from definition of dust. (random-zebra) a1c56fd [Policy] Introduce -dustrelayfee (random-zebra) 9fb29cc [Doc] Update multiwallet release notes (random-zebra) 379255e [Tests][Trivial] Add wallet_multiwallet.py to test_runner (random-zebra) 808fbc3 [Bugfix] consider boolean value of -zapwallettxes ParameterInteraction (random-zebra) f9141f8 [QA] Add wallet_multiwallet.py functional test (John Newbery) 2e02006 Rename -usewallet to -rpcwallet (Alex Morcos) a64440b Select wallet based on the given endpoint (Jonas Schnelli) 5683a9c Complete previous commit by moving mn stuff out of libbitcoin_wallet (random-zebra) b0fe92f Fix test_pivx circular dependency issue (random-zebra) 6cb2b92 Add wallet endpoint support to bitcoin-cli (-usewallet) (Jonas Schnelli) 7dd3916 Register wallet endpoint (Jonas Schnelli) 5bd1bd7 Properly forbid -salvagewallet and -zapwallettxes for multi wallet. (Alex Morcos) 41a7335 Remove unused variables (practicalswift) 5c3d73f Avoid CWalletTx copies in GetAddressBalances and GetAddressGroupings (Russell Yanofsky) e7cafab [Refactoring] Mimic ListCoins for sapling notes (random-zebra) 54fa122 [qt] Move some WalletModel functions into CWallet (random-zebra) 494ba64 [test] Add test for getmemoryinfo (random-zebra) 2394083 [Wallet] unset change position when there is no change on exact match (Gregory Sanders) 7d977ac Remove unused C++ code not covered by unit tests (random-zebra) 60bb4da ApproximateBestSubset should take inputs by reference, not value (Ryan Havar) 3633d75 Initialize nRelockTime (Patrick Strateman) 3a599d0 [Refactor] Return safeTx boolean in CheckTXAvailability (random-zebra) f219be9 Add safe flag to listunspent result (NicolasDorier) 0201065 Add COutput::fSafe member for safe handling of unconfirmed outputs (random-zebra) 75c8c6d Disallow copy of CReserveKeys (Gregory Sanders) 8378322 [Refactor] Replace optional reserveKey in PBF with unique pointer (random-zebra) Pull request description: I think these are all the remaining Bitcoin Core v0.15 PRs in the wallet area that we don't have yet, and are useful to us. I've grouped them here since they are all pretty small, simple, and narrow-focused (on the wallet/rpcwallet files). This changeset is based on top of - [x] #2337 as it keeps going with the multiwallet implementation, adding: - RPC endpoint support - `listwallets` RPC - wallet name in `getwalletinfo` RPC - `wallet_multiwallet.py` functional test As in some areas we are much closer to upstream, some of the commits needed adaptations (especially the functional tests). A couple of commits have been extended to include shield-related code. List of upstream PRs backported/adapted: - bitcoin#9906 : Disallow copy constructor CReserveKeys - bitcoin#9830 : Add safe flag to listunspent result - bitcoin#9993 : Initialize nRelockTime - bitcoin#10108 : ApproximateBestSubset should take inputs by reference, not value - bitcoin#10075 : Remove unused C++ code not covered by unit tests - bitcoin#10257 : Add test for getmemoryinfo - bitcoin#10295 : Move some WalletModel functions into CWallet - bitcoin#10500 : Avoid CWalletTx copies in GetAddressBalances and GetAddressGroupings - bitcoin#10522 : Remove unused variables - bitcoin#10816 : Properly forbid -salvagewallet and -zapwallettxes for multi wallet - bitcoin#10849 : Multiwallet: simplest endpoint support - bitcoin#9380 : Separate different uses of minimum fees (only dustRelayFee) - bitcoin#10817 : Redefine Dust and add a discard_rate (only first commit) - bitcoin#10883 : Rename -usewallet to -rpcwallet - bitcoin#10604 : Add listwallets RPC, include wallet name in getwalletinfo + tests - bitcoin#10893 : Avoid running multiwallet.py twice - bitcoin#10870 : Use wallet 0 in rpc console if running with multiple wallets - bitcoin#10931 : Fix misleading "method not found" multiwallet errors - bitcoin#10885 : Reject invalid wallets - bitcoin#11022 : Basic keypool topup - bitcoin#10976 : [MOVEONLY] Move some static functions out of wallet.h/cpp - bitcoin#11310 : Test listwallets RPC - bitcoin#10858 : "errors" to getblockchaininfo + unify "errors" field in get*info RPCs - bitcoin#11492 : Fix leak in CDB constructor - bitcoin#11476 : Avoid opening copied wallet databases simultaneously - bitcoin#11590 : always show help-line of wallet encryption calls - bitcoin#11289 : Add wallet backup text to import* and add* RPCs ACKs for top commit: furszy: re-re-utACK d5526bd. Fuzzbawls: ACK d5526bd Tree-SHA512: 115c4916ee212539b0a1b2cb25783ddf6753f5376de739a084191e874ed8717fff2c7cd9d744e397891f14eaa459ef2f48d675168621ef7316e81758fa6559f2

656fde5 Add script birthtime metadata to dump and import wallet (MeshCollider) 1bab9b2 Add script dump note to RPC help text and release notes (MeshCollider) 68c1e00 Add test for importwallet (MeshCollider) 9e1184d Add dumpwallet scripts test (MeshCollider) ef0c730 Add scripts to importwallet RPC (MeshCollider) b702ae8 Add CScripts to dumpwallet RPC (MeshCollider) cdc260a Add GetCScripts to CBasicKeyStore (MeshCollider) Pull request description: As discussed in bitcoin#11289 (comment), adds the CScripts from the wallet to the `dumpwallet` RPC and then allows them to be imported with the `importwallet` RPC. Includes a basic test, and modifies the helptext of the dumpwallet RPC. Notes: - Reviewers: use `?w=1` to avoid the indentation-only change in commit `Add scripts to importwallet RPC ` - currently the scripts are followed with `# addr=` comments just as the other keys are, unsure if this might confuse users into thinking all the scripts are for valid P2SH addresses though, but I don't think that should be an issue. - there are no birthtimes for scripts, so script imports don't affect rescans - `importwallet` imports the CScripts but I'm not sure how to approach specifying whether scripts are for P2SH addresses, BIP173 addresses, etc. whether that matters or not. Otherwise the RPC helptext might just need modification. Fixes bitcoin#11715 Tree-SHA512: 36c55837b3a58b9d3499d4c0c2ae82153d62aa71919e751574651b63a1d2b8ecc83796db4553cc65dad9b5341c3a42ae2fcf4d62598c30af267f8e1461ba8272

fanquake added the RPC/REST/ZMQ label Sep 8, 2017

achow101 reviewed Sep 8, 2017

View reviewed changes

maflcko added this to the 0.15.1 milestone Sep 10, 2017

maflcko added Docs Needs backport labels Sep 10, 2017

laanwj mentioned this pull request Sep 11, 2017

Make sure ~/.bitcoin doesn't exist before build #11297

Merged

TheBlueMatt reviewed Sep 21, 2017

View reviewed changes

mess110 reviewed Sep 27, 2017

View reviewed changes

Add wallet backup text to import*, add* and dumpwallet RPCs

a38bfbc

TheBlueMatt reviewed Oct 17, 2017

View reviewed changes

Wrap dumpwallet warning and note scripts aren't dumped

c098c58

JohnVonNeumann mentioned this pull request Nov 7, 2017

add* RPCs require backups #11243

Closed

JohnVonNeumann reviewed Nov 7, 2017

View reviewed changes

laanwj merged commit c098c58 into bitcoin:master Nov 8, 2017

maflcko removed the Needs backport label Nov 9, 2017

maflcko removed this from the 0.15.2 milestone Nov 9, 2017

maflcko pushed a commit to maflcko/bitcoin-core that referenced this pull request Nov 9, 2017

Add wallet backup text to import*, add* and dumpwallet RPCs

42ea47d

Github-Pull: bitcoin#11289 Rebased-From: a38bfbc

maflcko pushed a commit to maflcko/bitcoin-core that referenced this pull request Nov 9, 2017

Wrap dumpwallet warning and note scripts aren't dumped

3f1db56

Github-Pull: bitcoin#11289 Rebased-From: c098c58

meshcollider mentioned this pull request Nov 12, 2017

Add scripts to dumpwallet RPC #11667

Merged

sipa mentioned this pull request Dec 6, 2017

When running bitcoind I keep getting - boost::filesystem::space: Operation not permitted #11825

Closed

random-zebra mentioned this pull request Apr 28, 2021

[Wallet] Laggard wallet-related backports from btc 0.15 PIVX-Project/PIVX#2351

Merged

1 task

bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021

Add wallet backup text to import* and add* RPCs #11289

Add wallet backup text to import* and add* RPCs #11289

Uh oh!

Conversation

meshcollider commented Sep 8, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

achow101 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

maflcko commented Sep 9, 2017

Uh oh!

meshcollider commented Sep 9, 2017

Uh oh!

meshcollider commented Sep 11, 2017

Uh oh!

achow101 commented Sep 15, 2017

Uh oh!

TheBlueMatt left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mess110 Sep 27, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

meshcollider commented Sep 30, 2017

Uh oh!

TheBlueMatt commented Oct 2, 2017

Uh oh!

meshcollider commented Oct 2, 2017

Uh oh!

TheBlueMatt commented Oct 4, 2017

Uh oh!

meshcollider commented Oct 4, 2017

Uh oh!

laanwj commented Oct 5, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

TheBlueMatt commented Oct 5, 2017

Uh oh!

meshcollider commented Oct 5, 2017

Uh oh!

meshcollider commented Oct 13, 2017

Uh oh!

TheBlueMatt left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

meshcollider commented Oct 19, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

meshcollider Nov 8, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

meshcollider commented Sep 8, 2017 •

edited

Loading

mess110 Sep 27, 2017 •

edited

Loading

laanwj commented Oct 5, 2017 •

edited

Loading

TheBlueMatt left a comment •

edited

Loading

meshcollider Nov 8, 2017 •

edited

Loading