#301: Generalize pubkey checksum display and check
pubkey:checksum format is displayed and can be passed to all commands. The checksum is validated.
Splits check_public_key()
function into three functions:
is_pubkey_and_check()
check_pubkey_formet()
validate_checksum()
gen_checksum()
Create display_pubkey_with_checksum()
function
Creates and update related tests.
Close #301 (closed), #320 (closed).
Merge request reports
Activity
changed milestone to %0.8.x
changed milestone to %0.8.0
added Authentication Feature Security labels
changed title from #301 (closed): Display the pubkey checksum everywhere to #301 (closed): Generalize pubkey’s checksum display
- Resolved by matograine
- Resolved by matograine
- Resolved by Moul
added 8 commits
- a43955f2 - [mod] #301 (closed) : create a function gen_checksum
- 4c513422 - [enh] #301 (closed) display the checksum in the tx command
- 8db3eb7c - [enh] #301 (closed) : make `history` command use <pubkey:checksum> format
- 99c0f23c - [enh] #301 (closed) make `balance` use <pubkey:checksum> format
- d38b80e8 - [enh] #301 (closed) : make `wot` and `id` commands use <pubkey:checksum> format
- 7c751558 - [enh] #301 (closed) : make `cert` command display <pubkey:checksum> format
- 389d92ed - [enh] #301 (closed) : make `membership` command display <pubkey:checksum> format
- b5c4c6c8 - [enh] #301 (closed) : make `authfile` command display <pubkey:checksum> format
Toggle commit list- Resolved by Moul
- Resolved by Moul
- Resolved by Moul
added 9 commits
- 318bed81 - [mod] #301 (closed) : create a function gen_checksum
- bf98cab1 - [mod] #301 (closed) : create function pubkey_with_checksum()
- 32d6d670 - [enh] #301 (closed) display the checksum in the tx command
- 396e62b8 - [enh] #301 (closed) : make `history` command use <pubkey:checksum> format
- 6863873e - [enh] #301 (closed) make `balance` use <pubkey:checksum> format
- fa0f74d0 - [enh] #301 (closed) : make `wot` and `id` commands use <pubkey:checksum> format
- 11553419 - [enh] #301 (closed) : make `cert` command display <pubkey:checksum> format
- 54d32123 - [enh] #301 (closed) : make `membership` command display <pubkey:checksum> format
- 3b4005ca - [enh] #301 (closed) : make `authfile` command display <pubkey:checksum> format
Toggle commit listadded 9 commits
- 4495c883 - [mod] #301 (closed) : create a function gen_checksum
- 7a5e4e38 - [mod] #301 (closed) : create function pubkey_with_checksum()
- fac7b2b2 - [enh] #301 (closed) display the checksum in the tx command
- 4a5b11e7 - [enh] #301 (closed) : make `history` command use <pubkey:checksum> format
- 21b21873 - [enh] #301 (closed) make `balance` use <pubkey:checksum> format
- f93d429c - [enh] #301 (closed) : make `wot` and `id` commands use <pubkey:checksum> format
- 163d5134 - [enh] #301 (closed) : make `cert` command display <pubkey:checksum> format
- 1e45b19f - [enh] #301 (closed) : make `membership` command display <pubkey:checksum> format
- 9528d83c - [enh] #301 (closed) : make `authfile` command display <pubkey:checksum> format
Toggle commit list@moul, just to be sure, can I merge ?
- Resolved by Moul
added 10 commits
- 0177e35a - [fix] #343 (closed) : allow pubkeys between 39 and 44 characters
- d13cfe88 - [mod] #301 (closed) : create a function gen_checksum
- d2e9d9ea - [mod] #301 (closed) : create function pubkey_with_checksum()
- a12d2bf3 - [enh] #301 (closed) display the checksum in the tx command
- 8d98fe39 - [enh] #301 (closed) : make `history` command use <pubkey:checksum> format
- bfb95677 - [enh] #301 (closed) make `balance` use <pubkey:checksum> format
- 193cfeeb - [enh] #301 (closed) : make `wot` and `id` commands use <pubkey:checksum> format
- 1fbe7803 - [enh] #301 (closed) : make `cert` command display <pubkey:checksum> format
- ff75a507 - [enh] #301 (closed) : make `membership` command display <pubkey:checksum> format
- 30d63301 - [enh] #301 (closed) : make `authfile` command display <pubkey:checksum> format
Toggle commit list- Resolved by Moul
added 10 commits
- 4f1babb5 - [fix] #343 (closed) : allow pubkeys between 39 and 44 characters.
- 6daf17b6 - [mod] #301 (closed) : create a function gen_checksum
- 3dcc0820 - [mod] #301 (closed) : create function pubkey_with_checksum()
- b73d3664 - [enh] #301 (closed) display the checksum in the tx command
- 8f0475e7 - [enh] #301 (closed) : make `history` command use <pubkey:checksum> format
- 19f292d9 - [enh] #301 (closed) make `balance` use <pubkey:checksum> format
- a6ff63d4 - [enh] #301 (closed) : make `wot` and `id` commands use <pubkey:checksum> format
- 18006d0c - [enh] #301 (closed) : make `cert` command display <pubkey:checksum> format
- bc947628 - [enh] #301 (closed) : make `membership` command display <pubkey:checksum> format
- 65387b6b - [enh] #301 (closed) : make `authfile` command display <pubkey:checksum> format
Toggle commit listchanged title from #301 (closed): Generalize pubkey’s checksum display to #301 (closed): Generalize pubkey checksum display
changed title from #301 (closed): Generalize pubkey checksum display to #301 (closed): Generalize pubkey checksum display and check
- Resolved by Moul
- Resolved by Moul
- Resolved by Moul
- Resolved by Moul
- Resolved by Moul
- Resolved by Moul
- Resolved by Moul
- Resolved by Moul
mentioned in issue #344 (closed)
- Resolved by Moul
- Resolved by Moul
added 14 commits
-
65387b6b...19896c30 - 2 commits from branch
dev
- da2c347c - [test] #307 (closed): Create test_tui.py file
- 8db00d8c - [fix] #343 (closed) : allow pubkeys between 39 and 44 characters.
- 77c5ef51 - [mod] #301 (closed) : create a function gen_checksum
- 2c72d0c4 - [mod] #237 (closed) #301 (closed) change check_public_key() behavior
- 60af2ae8 - [mod] #301 (closed) : create function pubkey_with_checksum()
- f1075237 - [enh] #301 (closed) display the checksum in the tx command
- af8076aa - [enh] #301 (closed) : make `history` command use <pubkey:checksum> format
- a3e975dc - [enh] #301 (closed) make `balance` use <pubkey:checksum> format
- 977a7c3c - [enh] #301 (closed) : make `wot` and `id` commands use <pubkey:checksum> format
- 920d0b63 - [enh] #301 (closed) : make `cert` command use <pubkey:checksum> format
- 7154f14f - [enh] #301 (closed) : make `membership` command display <pubkey:checksum> format
- b34c3987 - [enh] #301 (closed) : make `authfile` command display <pubkey:checksum> format
Toggle commit list-
65387b6b...19896c30 - 2 commits from branch
- Resolved by Moul
- Resolved by Moul
Interesting thing : If you use
wot
with a part of someone's uid, silkaj answers.$ silkaj wot atogr Matograine (CmFKubyq…:CQ5) from block #106433-00000340… received 11 and sent 25/100 certifications: ...
I think it is a cool feature. There is no problem leaving it as it is, since a choice will be asked if there are many answers.
$ silkaj wot ato | id | uid | pubkey | timestamp | |------+-----------------+--------------------------------------------------+-----------------------| | 00 | 13megatonnes | hmQxjHC9Vn173CX4UaSpqiqjDXY5iqD41yeJcUsgGBZ:63c | 362032-00000016C2547… | | 10 | alouatouf77 | sieUwCWvHKzDLf2hBc3u8shTFPJ9uGpgJKFNLSmdwit:D7m | 362177-0000001006228… | | 20 | Duraton | 9ad856NFt5kNqUkUzAXeBhGVAoJoZ1ATWqQvgjCzJRUP:9jo | 332446-0000005396EEA… | | 30 | Fiatoux | 5WD4WSHE96ySreSwQFXPqaKaKcwboRNApiPHjPWB6V9C:EnZ | 0-E3B0C44298FC1C149A… | ...
It also works with parts of pubkeys :
$ silkaj wot FhgFDYqwcKgG enoratatouille (FhgFDYqw…:2W2) from block #205549-0000042A… received 5 and sent 2/100 certifications: ...
Edited by matograine- Resolved by Moul
- Resolved by Moul
added 13 commits
-
c7e0069c - 1 commit from branch
dev
- a1c26b73 - [test] #307 (closed): Create test_tui.py file
- 8770355b - [fix] #343 (closed) : allow pubkeys between 39 and 44 characters.
- 47650302 - [mod] #301 (closed) : create a function gen_checksum
- 8350b987 - [mod] #320 (closed) #301 (closed) change check_public_key() behavior
- 97daa0c9 - [mod] #301 (closed) : create function pubkey_with_checksum()
- 917db558 - [enh] #301 (closed) display the checksum in the tx command
- 964a09c0 - [enh] #301 (closed) : make `history` command use <pubkey:checksum> format
- 1f4ebf20 - [enh] #301 (closed) make `balance` use <pubkey:checksum> format
- 5f8a6a1a - [enh] #301 (closed) : make `wot` and `id` commands use <pubkey:checksum> format
- 3ca9140d - [enh] #301 (closed) : make `cert` command use <pubkey:checksum> format
- d3bef238 - [enh] #301 (closed) : make `membership` command display <pubkey:checksum> format
- 51d077f1 - [enh] #301 (closed) : make `authfile` command display <pubkey:checksum> format
Toggle commit list-
c7e0069c - 1 commit from branch
- Resolved by Moul