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
Fix whitelistpositivecase
test in LeakyContractTestsSuite
#9955
Conversation
…ermittedDelegateCallers and use it to check for qualified delegate callers. Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
Node: E2E Test Results 1 files ± 0 1 suites ±0 21m 50s ⏱️ + 21m 50s Results for commit b916917. ± Comparison against base commit edc657a. This pull request removes 1 and adds 310 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
# Conflicts: # hedera-node/hedera-config/src/main/java/module-info.java
Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #9955 +/- ##
=============================================
+ Coverage 65.27% 65.28% +0.01%
- Complexity 30202 30211 +9
=============================================
Files 3329 3329
Lines 126937 126940 +3
Branches 13198 13200 +2
=============================================
+ Hits 82853 82872 +19
+ Misses 40868 40852 -16
Partials 3216 3216 ☔ View full report in Codecov by Sentry. |
...vice-impl/src/main/java/com/hedera/node/app/service/contract/impl/exec/utils/FrameUtils.java
Outdated
Show resolved
Hide resolved
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.
I think it's likely easiest to make this config property return a Set<Long>
and then limit our exemptions in HtsSystemContract
to just long zero addresses whose 0.0.X
number is in that set.
…et<Long> for permittedDelegateCallers and converting it in the check in FrameUtils.java Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
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.
LGTM, very nice work @MrValioBg !
Description:
Fix
whitelistpositivecase
test in LeakyContractTestsSuite.Add support for besu Address in hedera-config in order to enable permittedDelegateCallers and add check in FrameUtils for qualified delegate callers.
Related issue(s):
Fixes #9407
Checklist