[Closed by !2722 which includes it] Use GetConsensusSigners for pre-confirmation-counting observed txin consensus slashing as well
[V133-specific]
[Closed by !2722 (merged)
'Slash non-signers upon consensus'
which includes it.]
Intended to close #1639 (closed)
'!2994's implementation of GetConsensusSigners is incomplete'.
Edit: This is also now closable by !2722 (merged) 'Slash non-signers upon consensus'.
For more details:
handler_observed_txin.go already uses voter.GetConsensusSigners()... for after-confirmation-count consensus unslashing ,
but still uses voter.Tx.GetSigners()... for before-confirmation-count consensus unslashing (which can miss transactions which contributed to that consensus while having minor differences from the Tx),
so the change made is to have both cases use voter.GetConsensusSigners()... .
Edited by Multipartite