Bitcoin Forum
December 08, 2016, 12:22:57 PM *
News: Latest stable version of Bitcoin Core: 0.13.1  [Torrent].
 
   Home   Help Search Donate Login Register  
Pages: « 1 [2]  All
  Print  
Author Topic: [PULL] monitortx monitorblocks listmonitored getblock  (Read 3869 times)
MrJoshua
Member
**
Offline Offline

Activity: 76


View Profile
August 10, 2011, 01:42:25 AM
 #21

From reading your other posts, I get the impression that you want to know about addresses that are not in the wallet.  I don't think this patch gives you that.

Oh doesn't it?  That was my understanding, as posted above, and no one has corrected me on that yet.

j

The value of bitcoins is not a theory, predictions of it's failure are what is theoretical.
1481199777
Hero Member
*
Offline Offline

Posts: 1481199777

View Profile Personal Message (Offline)

Ignore
1481199777
Reply with quote  #2

1481199777
Report to moderator
Advertised sites are not endorsed by the Bitcoin Forum. They may be unsafe, untrustworthy, or illegal in your jurisdiction. Advertise here.
1481199777
Hero Member
*
Offline Offline

Posts: 1481199777

View Profile Personal Message (Offline)

Ignore
1481199777
Reply with quote  #2

1481199777
Report to moderator
1481199777
Hero Member
*
Offline Offline

Posts: 1481199777

View Profile Personal Message (Offline)

Ignore
1481199777
Reply with quote  #2

1481199777
Report to moderator
kjj
Legendary
*
Offline Offline

Activity: 1302



View Profile
August 10, 2011, 02:57:08 AM
 #22

My C++ is really rusty, and I'm reading the diff out of context, but I believe that this patch just lets you add URLs to two lists, one that gets POSTs for every new block that comes in, and one that gets POSTs for every transaction that comes in related to the running wallet.

p2pcoin: a USB/CD/PXE p2pool miner - 1N8ZXx2cuMzqBYSK72X4DAy1UdDbZQNPLf - todo
I routinely ignore posters with paid advertising in their sigs.  You should too.
2112
Legendary
*
Offline Offline

Activity: 1708



View Profile
August 10, 2011, 04:13:32 AM
 #23

... but I'm not 100% happy with it. I'm not sure it properly handles block chain re-orgs and dependent orphan transactions. Would be nice to write some tests to exercise those edge cases, and figure out what it SHOULD do in those cases.
In addition to the above this patch doesn't seem to have any mechanism for garbage collecting stale notification callbacks. From the cursory look I would guess that it doesn't even try to be idempotent, i.e. one could add multiple identical callback URLs with no limit that I could see.

Please comment, critique, criticize or ridicule BIP 2112: https://bitcointalk.org/index.php?topic=54382.0
Long-term mining prognosis: https://bitcointalk.org/index.php?topic=91101.0
BitcoinLocator
Jr. Member
*
Offline Offline

Activity: 31


www.BitcoinLocator.com


View Profile WWW
August 11, 2011, 05:55:46 PM
 #24

My C++ is really rusty, and I'm reading the diff out of context, but I believe that this patch just lets you add URLs to two lists, one that gets POSTs for every new block that comes in, and one that gets POSTs for every transaction that comes in related to the running wallet.

I hope this is not the case, and it doesn't make sense.  If you're receiving blocks complete with all transactions and you have addresses in your POST queue for monitortx why would you go out of your way to filter results based on the wallet?  It's an extra step to reduce functionality.


Are you a BitcoinLocation? www.bitcoinlocator.com
Furyan
Full Member
***
Offline Offline

Activity: 175



View Profile
September 08, 2011, 12:22:27 PM
 #25

Gavin,

I have been running with this patch for a few weeks now (decided to go ahead and see how big the problem might be).

So, orphaned blocks received from the network definitely do get posted via the monitorBlock workflow.  However, when bitcoin eventually re-orgs the block chain to replace the orphaned blocks with the real block from the longest chain, those blocks are not being posted.  For now I have a process that runs once in awhile to detect these cases and updates the values stored in the db, but I'm wondering if something can be modified in the patch to address it.

Interestingly, if the orphaned block is one of my own, I actually want to retain the old value for tracking purposes.

With regard to dependent orphaned transactions, I haven't run into those yet so can't comment.
Pages: « 1 [2]  All
  Print  
 
Jump to:  

Sponsored by , a Bitcoin-accepting VPN.
Powered by MySQL Powered by PHP Powered by SMF 1.1.19 | SMF © 2006-2009, Simple Machines Valid XHTML 1.0! Valid CSS!