1 2017-12-10 00:56:13 0|andytoshi|sipa: ahh ok, thanks 2 2017-12-10 05:54:23 0|meshcollider|why is the boost::to_lower() inside a #ifdef WIN32 here: https://github.com/bitcoin/bitcoin/blob/master/src/util.cpp#L419 3 2017-12-10 05:54:41 0|meshcollider|does that mean its only case insensitive on win32 4 2017-12-10 05:57:20 0|Lightsword|windows itself is mostly case insensitive 5 2017-12-10 05:58:24 0|Lightsword|although I think some of the underlying filesystems used like NTFS can be case sensivite 6 2017-12-10 06:05:46 0|sipa|NTFS on windows is case insensitive but case preserving 7 2017-12-10 06:06:40 0|sipa|but that line of code has nothing do with filesystems 8 2017-12-10 06:07:09 0|sipa|it's so that "bitcoind /daemon=1" works, which I think gets passed to the application as "/DAEMON=1" 9 2017-12-10 06:47:54 0|meshcollider|ah I see 10 2017-12-10 07:15:51 0|bitcoin-git|[13bitcoin] 15irq13 opened pull request #11860: move banned peer log messages behind -debug=net (06master...06no-log-bans) 02https://github.com/bitcoin/bitcoin/pull/11860 11 2017-12-10 11:03:23 0|ieririr|hi there, does anybody have an idea of the time needed to spread a fix? in case of major flaw in crypto core I mean 12 2017-12-10 11:16:57 0|promag|is it me or funding an already funded transaction hits AvailableCoins? 13 2017-12-10 18:05:19 0|andytoshi|sipa: /win 23 14 2017-12-10 18:05:22 0|andytoshi|oops sorry 15 2017-12-10 19:30:06 0|BGL|is it by design that the core client gives no indication that it's creating a new wallet file (ie, when one goes missing) ? or is that a bug? 16 2017-12-10 20:47:01 0|sipa|BGL: what would you expect to happen? 17 2017-12-10 21:36:26 0|BGL|at a minimum say hey your wallet file was missing? 18 2017-12-10 21:37:23 0|sipa|i think it does say that in the log, but maybe not 19 2017-12-10 21:40:50 0|BGL|certain things have a real minimum-effort feel going for it with the client 20 2017-12-10 21:41:13 0|BGL|and with btc being 15k now i think the standards might need to be up a little 21 2017-12-10 21:41:23 0|sipa|what else would you expect? 22 2017-12-10 21:41:44 0|BGL|so like let's say your sql db goes down 23 2017-12-10 21:41:54 0|BGL|<insert variable of obvious common sense> 24 2017-12-10 21:42:09 0|BGL|asking what people expect is not lending toward resolution 25 2017-12-10 21:42:24 0|sipa|i don't understand what you feel is missing 26 2017-12-10 21:42:36 0|BGL|that may be exactly the problem 27 2017-12-10 21:43:36 0|sipa|okay 28 2017-12-10 21:43:47 0|sipa|well, issue reports or pull requests welcome 29 2017-12-10 21:44:22 0|Randolf|BGL: Are there some specific problems with the standards that you are aware of presently? 30 2017-12-10 21:44:29 0|BGL|waiting on people to point out the obvious is often the demise of software 31 2017-12-10 21:44:43 0|sipa|what do you expect to happen? fail to start? 32 2017-12-10 21:44:58 0|sipa|that would break backward compatibility 33 2017-12-10 21:45:23 0|BGL|yes, let's all act as if your wallet file disappearing and the client literally indicating nothing is wrong is perfectly acceptable. 34 2017-12-10 21:45:38 0|BGL|i would literally fire both of you for acting so stupid 35 2017-12-10 21:45:47 0|sipa|the client is unable to distinguish that from starting up the first time 36 2017-12-10 21:45:49 0|BGL|the correct answer is yes, maybe that's a good idea, we should look into that 37 2017-12-10 21:46:05 0|BGL|maybe that's part of the problem? 38 2017-12-10 21:46:12 0|BGL|does this sound like such a difficult issue? 39 2017-12-10 21:46:15 0|sipa|yes 40 2017-12-10 21:46:27 0|BGL|wow 41 2017-12-10 21:46:38 0|sipa|there is literally nothing the client could do the know that missing a file is different from starting up the first time 42 2017-12-10 21:46:42 0|Randolf|BGL: I'm not sure who you're directing that to, but please keep in mind that people here are volunteers. 43 2017-12-10 21:46:53 0|sipa|you're very welcome to help improve the software 44 2017-12-10 21:47:27 0|sipa|but insulting volunteers is not likely to get your issue resolved 45 2017-12-10 21:47:33 0|BGL|being a volunteer does not make up for incompetent ego 46 2017-12-10 21:47:46 0|BGL|my only issue is the quality of this client 47 2017-12-10 21:47:52 0|Randolf|BGL: With regard to people losing their wallet files, backups are highly recommended, and having copies of one's Private Keys is obviously imperative. Perhaps if this isn't being explained to the end-user properly there could indeed be room for improvement there. 48 2017-12-10 21:48:07 0|sipa|BGL: you must literally have manually deleted a database file 49 2017-12-10 21:48:26 0|sipa|there is no other way the file would be missing 50 2017-12-10 21:48:36 0|sipa|that is the exact same situation as starting up the first time 51 2017-12-10 21:48:42 0|sipa|please, be constructive 52 2017-12-10 21:48:56 0|Randolf|BGL: My ego doesn't factor into this -- it went on vacation a long time ago, and I have no idea where it is these days, although I do keep receiving hotel bills so I know where it has been. ;) 53 2017-12-10 21:49:29 0|Randolf|sipa: There is another possibility -- the user logged in with a different profile and so it looks like a new setup to the client software. 54 2017-12-10 21:49:31 0|gmaxwell|I've never previously heard a report from a user that their wallet file was unexpectedly removed. Any idea how a user would end up in that situation? 55 2017-12-10 21:49:52 0|Randolf|BGL: Did your wallet file go missing? 56 2017-12-10 21:50:06 0|sipa|Randolf: "profile" ? 57 2017-12-10 21:50:33 0|Randolf|sipa: Well, in the Windows world at least, there are user profiles which each have their own collections of home directories and personalized settings for the OS and various applications. 58 2017-12-10 21:51:39 0|Randolf|sipa: Unfortunately with Windows, whenever it corrupts itself the user is sometimes logged in with a temporary profile instead of their usual one. In such cases, a wallet file could seem to go missing if it was stored under the user-specific home directory. 59 2017-12-10 21:51:59 0|Randolf|sipa: But this really isn't a Bitcoin issue -- it's an Operating System issue that has the same impact on other applications too. 60 2017-12-10 21:52:35 0|gmaxwell|Randolf: how would that deny access to the wallet but also leave the blockchain? 61 2017-12-10 21:53:47 0|Randolf|gmaxwell: That's interesting. If the blockchain was also stored in the user's profile directory, then your concern makes sense. However, if the blockchain was stored in a public profile (as some applications with shared or constant data do) then that could explain why the blockchain would remain 62 2017-12-10 21:53:47 0|Randolf|in place. 63 2017-12-10 21:54:23 0|Randolf|gmaxwell: Another possibility is that bitcoind has its own profile and is storing the blockchain data under its own profile which is not the same as the user's profile. 64 2017-12-10 21:54:30 0|Randolf|...specifically the wallet-user's profile. 65 2017-12-10 21:54:34 0|sipa|Randolf: unless he's running unreleased software, the blockchain and the wallet are in the same directory 66 2017-12-10 21:54:45 0|Randolf|sipa: Okay, then that rules that possibility out. 67 2017-12-10 21:55:15 0|Randolf|Another possibility is that the user installed some malicious software that uploads wallet files then deletes them from the user's system. 68 2017-12-10 22:03:52 0|BGL|Randolf: that's not possible because <the user> "must literally have manually deleted a database file" 69 2017-12-10 22:05:22 0|sipa|BGL: if that's not what happened, i'd very much like to know 70 2017-12-10 22:07:48 0|BGL|please save me from having to explain an infinite number of ways files can be deleted 71 2017-12-10 22:08:31 0|sipa|not by bitcoind itself 72 2017-12-10 22:09:02 0|sipa|either you directly, or software you chose to run deleted a file 73 2017-12-10 22:09:22 0|BGL|reminder: i'm simply pointing out the client says/does nothing when a wallet file goes missing 74 2017-12-10 22:09:31 0|sipa|yes, because *it can't* 75 2017-12-10 22:15:10 0|mesh_|Best we could do is add a debug.log message which says something like "wallet.dat does not exist, creating..." 76 2017-12-10 22:15:33 0|mesh_|BGL: that could be an opportunity for you to contribute, you could open a PR if you wanted :) 77 2017-12-10 22:15:34 0|gmaxwell|mesh_: the logs do reflect when it creates one. 78 2017-12-10 22:16:24 0|gmaxwell|The software also does not warn you when your computer is running in an enviroment without enough oxygen for human life... there are an infinitude of ways that something strange could be going on; without an understood sequence of events that would cause the situation it's usually not reasonable to do anything about it, since the "do something" is usually context dependant. 79 2017-12-10 22:17:31 0|mryandao|^ lol. 80 2017-12-10 22:18:40 0|meshcollider|gmaxwell: I can only see a message like "2017-12-10 22:16:37 Using wallet test.dat" and then it reports there are 0 keys, but no actual creation message? 81 2017-12-10 22:19:22 0|meshcollider|https://www.irccloud.com/pastebin/rzTcABjN/ 82 2017-12-10 22:20:07 0|gmaxwell|Morover, spurrious warnings cause user error. E.g. we've seen users lose funds because they opened their debug.log and saw log entries with "ERROR" in them, and then started taking increasingly drastic actions to remediate the actually-harmless messages which eventually culminated in them forever deleting their wallet, or posting their files online for othe people ... including the wallet. 83 2017-12-10 22:20:13 0|gmaxwell|Moreover* 84 2017-12-10 22:20:46 0|gmaxwell|meshcollider: indeed, it shows it with zero sizes when its first created. 85 2017-12-10 22:21:45 0|mryandao|maybe bitcoin-core should build in telemetry service and leave the generation of debug log for advanced users. 86 2017-12-10 22:21:52 0|meshcollider|gmaxwell: yeah but that is not clear to any user, I'm just suggesting BGL might want to add a simpler message 87 2017-12-10 22:21:57 0|mryandao|to protect users against themselves 88 2017-12-10 22:25:54 0|gmaxwell|mryandao: well we intentionally toned down the text in debug log to reduce incidents of those sorts of screwups, seems to have owrked. 89 2017-12-10 22:26:15 0|gmaxwell|meshcollider: would be reasonable and harmless, though I'm dubious that it would actually help anything. 90 2017-12-10 22:41:06 0|Randolf|BGL: Doesn't the client report to you that the file was not found? 91 2017-12-10 22:41:20 0|Randolf|BGL: ...when you attempt to open it? 92 2017-12-10 22:42:03 0|sipa|Randolf: there is no 'opening' the file 93 2017-12-10 22:43:17 0|sipa|there is a hardcoded filename (wallet.dat) which contains the wallet; if it doesn't exist, it's created 94 2017-12-10 22:44:29 0|Randolf|sipa: Ah, well, in that case the client software won't be able to detect if a previous wallet file was deleted. 95 2017-12-10 22:44:32 0|sipa|the daemon supports having multiple wallets loaded simultaneously, but the GUI doesn't support opening/closing other wallets 96 2017-12-10 22:44:44 0|sipa|(not yet, at least) 97 2017-12-10 22:46:27 0|Randolf|Okay. 98 2017-12-10 22:47:36 0|Randolf|That's very curious. I wonder what happened to BGL's wallet file. I'm sure that neither the client nor bitcoind deleted it. 99 2017-12-10 22:47:52 0|Randolf|BGL: Did you save your Private Keys? 100 2017-12-10 22:48:02 0|Randolf|BGL: If you did, then you can enter them again.