ByteCoin
|
|
August 19, 2010, 05:18:45 PM |
|
I hope you have permission from the people in the converstation to post that. If not - bad form. If you edit your post to included "quoted with permission" I will delete this post
ByteCoin
|
|
|
|
aceat64
|
|
August 19, 2010, 07:21:11 PM |
|
Quoted with permission from me at least, so if need be mizerydearia can edit the post and make it a weird monologue.
|
|
|
|
mizerydearia
|
|
August 26, 2010, 01:20:46 AM |
|
I glanced at http://www.bitcoin.org/wiki/doku.php?id=list_of_patches and haven't noticed any updates for this. Is listtransactions established or useful/functional yet? Is there a binary release (official or unofficial) that integrates this patch that I could use? I would like to use the data extracted from using the listtransactions method to implement into my website if it is reliable to do so. Specifically in a current use scenario I would like to be able to determine the last transaction that occurred for a particular address label and to extract related data such as amount and date/time.
|
|
|
|
jgarzik (OP)
Legendary
Offline
Activity: 1596
Merit: 1099
|
|
August 26, 2010, 01:43:53 AM |
|
It has been functional since day one Is there a binary release (official or unofficial) that integrates this patch that I could use?
Not AFAIK.
|
Jeff Garzik, Bloq CEO, former bitcoin core dev team; opinions are my own. Visit bloq.com / metronome.io Donations / tip jar: 1BrufViLKnSWtuWGkryPsKsxonV2NQ7Tcj
|
|
|
aceat64
|
|
September 01, 2010, 03:47:06 PM |
|
Is there a binary release (official or unofficial) that integrates this patch that I could use?
I'm now doing binary releases with my github repo: http://github.com/aceat64/bitcoin-patchworkI've got the following patches included: I've also made the following changes: - Slight change so that JSON-RPC returns 200 instead of a 500 error with invalid methods/params.
- Slight change so that JSON-RPC binds to all/any IPs instead of just the loopback.
- Changed MAX_OUTBOUND_CONNECTIONS from 8 to 32.
|
|
|
|
Gavin Andresen
Legendary
Offline
Activity: 1652
Merit: 2301
Chief Scientist
|
|
September 01, 2010, 06:33:54 PM |
|
- Slight change so that JSON-RPC returns 200 instead of a 500 error with invalid methods/params.
- Slight change so that JSON-RPC binds to all/any IPs instead of just the loopback.
RE: JSON-RPC error / HTTP status codes: I'm about to submit a patch to Satoshi so bitcoin better follows the JSON standards. I'll create a new thread describing the changes. RE: binding to any/all IPs: DANGER! Make sure you have good firewall rules for port 8332 or you are likely to find yourself with an empty wallet! Bitcoin is using HTTP BASIC authentication, so anybody who can eavesdrop on the connection between your JSON-RPC client and the bitcoin server can see the password (it is NOT encrypted).
|
How often do you get the chance to work on a potentially world-changing project?
|
|
|
jgarzik (OP)
Legendary
Offline
Activity: 1596
Merit: 1099
|
|
September 01, 2010, 06:35:40 PM |
|
RE: binding to any/all IPs: DANGER! Make sure you have good firewall rules for port 8332 or you are likely to find yourself with an empty wallet! Bitcoin is using HTTP BASIC authentication, so anybody who can eavesdrop on the connection between your JSON-RPC client and the bitcoin server can see the password (it is NOT encrypted).
I was thinking about adding support for Digest auth...
|
Jeff Garzik, Bloq CEO, former bitcoin core dev team; opinions are my own. Visit bloq.com / metronome.io Donations / tip jar: 1BrufViLKnSWtuWGkryPsKsxonV2NQ7Tcj
|
|
|
jgarzik (OP)
Legendary
Offline
Activity: 1596
Merit: 1099
|
|
September 01, 2010, 06:40:17 PM |
|
Patch updated for current SVN, r146, aka the "break patches with a CR/LF policy change" revision.
Same patch URL (see top of this thread), no change in implementation or operation.
|
Jeff Garzik, Bloq CEO, former bitcoin core dev team; opinions are my own. Visit bloq.com / metronome.io Donations / tip jar: 1BrufViLKnSWtuWGkryPsKsxonV2NQ7Tcj
|
|
|
Gavin Andresen
Legendary
Offline
Activity: 1652
Merit: 2301
Chief Scientist
|
|
September 01, 2010, 07:35:27 PM |
|
RE: binding to any/all IPs: DANGER! Make sure you have good firewall rules for port 8332 or you are likely to find yourself with an empty wallet! Bitcoin is using HTTP BASIC authentication, so anybody who can eavesdrop on the connection between your JSON-RPC client and the bitcoin server can see the password (it is NOT encrypted).
I was thinking about adding support for Digest auth... I wouldn't bother-- HTTP Digest auth is vulnerable to man-in-the-middle attacks, it'd be a lot of work for a probably false sense of security.
|
How often do you get the chance to work on a potentially world-changing project?
|
|
|
mizerydearia
|
|
September 01, 2010, 08:20:16 PM |
|
Is there a binary release (official or unofficial) that integrates this patch that I could use?
I'm now doing binary releases with my github repo: http://github.com/aceat64/bitcoin-patchworkI've got the following patches included: I've also made the following changes: - Slight change so that JSON-RPC returns 200 instead of a 500 error with invalid methods/params.
- Slight change so that JSON-RPC binds to all/any IPs instead of just the loopback.
- Changed MAX_OUTBOUND_CONNECTIONS from 8 to 32.
Can you provide link to patches for listgenerated method and the following three changes at the end? I'd like to incorporate them into my Gentoo Linux ebuild.
|
|
|
|
jgarzik (OP)
Legendary
Offline
Activity: 1596
Merit: 1099
|
|
September 01, 2010, 08:36:28 PM |
|
RE: binding to any/all IPs: DANGER! Make sure you have good firewall rules for port 8332 or you are likely to find yourself with an empty wallet! Bitcoin is using HTTP BASIC authentication, so anybody who can eavesdrop on the connection between your JSON-RPC client and the bitcoin server can see the password (it is NOT encrypted).
I was thinking about adding support for Digest auth... I wouldn't bother-- HTTP Digest auth is vulnerable to man-in-the-middle attacks, it'd be a lot of work for a probably false sense of security. For Some Value Of vulnerable. The password cannot be obtained, at least. MiTM can (a) tell the client to use another auth, and (b) impersonate a server. (a) can be mitigated by requiring RFC 2617, (b) by SSL. But, use of HTTP-Basic is just a crime, because it is so trivial to obtain the shared secret. If HTTP-Basic is to be kept, at least require SSL connections?
|
Jeff Garzik, Bloq CEO, former bitcoin core dev team; opinions are my own. Visit bloq.com / metronome.io Donations / tip jar: 1BrufViLKnSWtuWGkryPsKsxonV2NQ7Tcj
|
|
|
aceat64
|
|
September 01, 2010, 09:21:33 PM |
|
Can you provide link to patches for listgenerated method and the following three changes at the end? I'd like to incorporate them into my Gentoo Linux ebuild.In the repo I have a folder called "patches" where I keep a copy of all the patches I apply.
|
|
|
|
jgarzik (OP)
Legendary
Offline
Activity: 1596
Merit: 1099
|
|
September 07, 2010, 07:21:30 AM |
|
Patch updated for SVN r147, which introduces trivial breakage w/ listtransactions. (see start of thread for URL)
|
Jeff Garzik, Bloq CEO, former bitcoin core dev team; opinions are my own. Visit bloq.com / metronome.io Donations / tip jar: 1BrufViLKnSWtuWGkryPsKsxonV2NQ7Tcj
|
|
|
mizerydearia
|
|
September 15, 2010, 08:39:10 PM |
|
I've also made the following changes: - Slight change so that JSON-RPC binds to all/any IPs instead of just the loopback.
This patch doesn't seem to be compatible with this patch. Can these patches be combined or is one better than the other?
|
|
|
|
jgarzik (OP)
Legendary
Offline
Activity: 1596
Merit: 1099
|
|
September 18, 2010, 07:44:33 PM |
|
Patch updated to latest SVN.
|
Jeff Garzik, Bloq CEO, former bitcoin core dev team; opinions are my own. Visit bloq.com / metronome.io Donations / tip jar: 1BrufViLKnSWtuWGkryPsKsxonV2NQ7Tcj
|
|
|
jgarzik (OP)
Legendary
Offline
Activity: 1596
Merit: 1099
|
|
September 29, 2010, 05:32:52 PM |
|
Patch updated to latest SVN, and for a change suggested by gavin: - sort by time, after sorting by number of confirmations
|
Jeff Garzik, Bloq CEO, former bitcoin core dev team; opinions are my own. Visit bloq.com / metronome.io Donations / tip jar: 1BrufViLKnSWtuWGkryPsKsxonV2NQ7Tcj
|
|
|
davout
Legendary
Offline
Activity: 1372
Merit: 1008
1davout
|
|
October 18, 2010, 01:06:15 PM |
|
Hello all !
Is this patch ever going to hit the main official vanilla client ?
I don't really get how it could be the 'wrong way' to track transfers for a web application as long as each user has a distinct address which really seems like an obvious pre-requisite for any kind of app that is going to support multiple users.
Without that patch it's not really possible for an user to have a single entry point for transfers to his account without doing some dangerous guesswork. And since, AFAIK, nothing prevents re-using by mistake a payment address that has already been used there is the possibility that some transferred funds will end up in some no man's land :/
|
|
|
|
teppy
|
|
October 20, 2010, 02:35:39 PM |
|
jgarzik,
Could you have listtransactions also include the comment field associated with a transaction? Here's the problem I'm running into:
Dragon's Tale MMO does periodic (every 3 minutes) checkpoints. If someone has withdrawn BTC since the last checkpoint and the server crashes, then when it comes back up (restoring the most recent checkpoint), they could withdraw the same BTC again!
I deal with the opposite case cleanly: When someone requests an address to use to deposit BTC, I label that address and use getreceivedbylabel when computing their balance: My server and bitcoind are always in agreement WRT how many BTC the person has deposited. But unless I can query bitcoind about how many BTC were sent out on behalf of a player, I just have to trust the value stored in the last checkpoint file - very dangerous!
My solution is to record a comment consisting of the player's name, in each outbound transaction, and then when doing a restore, read in the last N transactions to correct for unrecorded outbound transactions in the game's database.
FWIW, this is almost exactly the same technique we use to sync the game database and our credit card merchant's database in A Tale in the Desert.
|
Dragon's Tale is the longest running Bitcoin enterprise in the world.
|
|
|
jgarzik (OP)
Legendary
Offline
Activity: 1596
Merit: 1099
|
|
October 21, 2010, 07:36:24 PM |
|
Patch updated to latest SVN (v0.3.14). Could you have listtransactions also include the comment field associated with a transaction? Here's the problem I'm running into:
Yeah, I think this is doable. Will look into it.
|
Jeff Garzik, Bloq CEO, former bitcoin core dev team; opinions are my own. Visit bloq.com / metronome.io Donations / tip jar: 1BrufViLKnSWtuWGkryPsKsxonV2NQ7Tcj
|
|
|
jgarzik (OP)
Legendary
Offline
Activity: 1596
Merit: 1099
|
|
November 13, 2010, 07:20:58 PM |
|
Patch updated to latest SVN. See top of thread for URL.
|
Jeff Garzik, Bloq CEO, former bitcoin core dev team; opinions are my own. Visit bloq.com / metronome.io Donations / tip jar: 1BrufViLKnSWtuWGkryPsKsxonV2NQ7Tcj
|
|
|
|