Pass os, gui to the Alert system #1005

issue gavinandresen opened this issue on March 28, 2012

gavinandresen commented at 11:05 PM on March 28, 2012: contributor

The client id in 'version' messages are specifically vague so potential attackers can't easily tell exactly what flavor of Bitcoin you're running.

But we can, and should, pass full information to the Alert system, so we can send alerts targeted to just Windows Bitcoin-Qt users.

I think just platform (Linux/OSX/Windows) and Qt-or-not.
laanwj commented at 6:24 AM on March 29, 2012: member

Good idea. Alerts should be as precisely targetable as possible.
Diapolo commented at 6:42 AM on March 29, 2012: none

Where is that code located? I only saw something uncommented from the source, which mentioned an alert system.
laanwj commented at 11:23 AM on March 29, 2012: member

@diapolo main.cpp: GetWarnings, ProcessAlert
Diapolo commented at 10:06 PM on April 12, 2012: none

Is there a good place to find a documentation on the code-flow, used classes, functions and how it works?
laanwj commented at 11:32 AM on October 27, 2015: member

The current alert system is pretty much deprecated (see discussions in #6260, #5160), certainly for reporting client software specific issues, so there is no point in doing this anymore
laanwj closed this on Oct 27, 2015
suprnurd referenced this in commit c269018016 on Dec 5, 2017
ptschip referenced this in commit f1e16fe031 on Apr 4, 2018
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels