False positives in `-alertnotify` #7488

issue laanwj opened this issue on February 9, 2016
  1. laanwj commented at 9:28 AM on February 9, 2016: member

    Services are using -alertnotify to be notified of critical problems. Some have pagers connected, or even shut off the service automatically.

    Some of the messages based on heuristics, such as the "abnormally high number of blocks" one seem to appear a lot despite there being no real issue: https://www.reddit.com/r/Bitcoin/comments/3ydwg2/warning_abnormally_high_number_of_blocks/

    As well as wasting time and resources, after the zillionth time users start ignoring the messages completely, and thus miss a serious problem.

    Another issue is that some of the warnings don't go away when the (temporary) problem goes away, the only way to turn them off is to restart bitcoind.

  2. laanwj added the label RPC on Feb 9, 2016
  3. laanwj renamed this:
    False positives in `-alertnotfy`
    False positives in `-alertnotify`
    on Feb 9, 2016
  4. petertodd commented at 12:43 PM on February 9, 2016: contributor

    Agreed - if this isn't fixed soon we should just remove it to avoid the false-positive effect.

    Also, I'm kinda unconvinced about the rational for the "blocks too fast" alert - the definition of Bitcoin is that you're on the most-work valid chain after all. Hard to think of cases where you'd actually see that in an attack, at least after you've synced up the first time.

  5. dcousens commented at 1:24 PM on February 9, 2016: contributor

    @petertodd maybe those who are concerned about breaking SHA256. /s You're right though, it does seem a bit unnecessary.

  6. sipa commented at 1:28 PM on February 9, 2016: member

    I assume the reasoning was: if hashrate slowly moves to a different chain, your difficulty drops, and then that hashrate returns suddenly, it's a sign of an economic situation that isn't very secure.

  7. laanwj commented at 5:29 PM on February 9, 2016: member

    Related: Falsely triggered "check your network connection" error while catching up. #6251

  8. laanwj added this to the milestone 0.13.0 on Apr 28, 2016
  9. laanwj removed this from the milestone 0.13.0 on Jul 7, 2016
  10. laanwj commented at 6:55 PM on July 7, 2016: member

    The most urgent part of this was solved with #8275, removing the milestone

  11. laanwj closed this on Jan 26, 2018

  12. DrahtBot locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-13 15:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me