-
Notifications
You must be signed in to change notification settings - Fork 277
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
GaroRobe/issue1485 #1517
GaroRobe/issue1485 #1517
Conversation
c6ecb01
to
da39dc0
Compare
Codecov Report
@@ Coverage Diff @@
## iroha2-dev #1517 +/- ##
==============================================
- Coverage 76.47% 76.36% -0.12%
==============================================
Files 112 112
Lines 17826 17868 +42
==============================================
+ Hits 13633 13645 +12
- Misses 4193 4223 +30
Continue to review full report at Codecov.
|
Signed-off-by: Artem Ponomarev <ponomarev@soramitsu.co.jp>
be3cb4c
to
0dbcd21
Compare
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.
Are these really ok though?
As you concerned it looks to me not to represent OnlyAccountsDomain
nor OnlyAccountsData
permissions.
I guess using random hashes everyone could know about some transactions related to other domain or account
@@ -376,6 +376,7 @@ pub mod private_blockchain { | |||
)) | |||
} | |||
} | |||
FindTransactionByHash(_query) => Ok(()), |
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.
OnlyAccountsDomain
@@ -542,6 +543,7 @@ pub mod private_blockchain { | |||
Err(format!("Cannot access another account: {}.", account_id)) | |||
} | |||
} | |||
FindTransactionByHash(_query) => Ok(()), |
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.
OnlyAccountsData
I agree the possibility of success in random access by hashes is very low. |
Description of the Change
Added a new query - FindTransactionByHash. Will search a confirmed or rejected transaction by hash value.
Benefits
Possible Drawbacks
Doesn't account for any permissions or domains. Which might be wrong or need a setting.
Usage Examples or Tests [optional]
Alternate Designs [optional]