-
Notifications
You must be signed in to change notification settings - Fork 366
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
INDY-2316 Allow multiple active TAAs: Debug #1436
Conversation
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
…nto task-2297-get-taa-digest
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
Signed-off-by: toktar <renata.toktar@dsr-corporation.com>
This pull request introduces 1 alert when merging a143655 into f2d582b - view on LGTM.com new alerts:
|
This pull request introduces 2 alerts when merging b3883b9 into f2d582b - view on LGTM.com new alerts:
|
@@ -46,7 +46,7 @@ def check_audit_txn(txn, | |||
"txn": { | |||
"data": { | |||
"ledgerRoot": expectedLedgerRoots, | |||
"ver": CURRENT_TXN_VERSIONS[AUDIT], | |||
"ver": CURRENT_TXN_PAYLOAD_VERSIONS[AUDIT], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it correct?
This pull request introduces 2 alerts when merging 39c6121 into f2d582b - view on LGTM.com new alerts:
|
@@ -83,7 +83,7 @@ def _create_audit_txn_data(self, three_pc_batch, last_audit_txn): | |||
# 1. general format and (view_no, pp_seq_no) | |||
view_no = three_pc_batch.original_view_no if three_pc_batch.original_view_no is not None else three_pc_batch.view_no | |||
txn = { | |||
TXN_VERSION: CURRENT_TXN_VERSIONS[AUDIT], | |||
TXN_VERSION: CURRENT_TXN_VERSION, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CURRENT_TXN_PAYLOAD_VERSIONS needs to be used here
This pull request introduces 2 alerts when merging 89dc75f into f2d582b - view on LGTM.com new alerts:
|
Changes: