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
Move token revoke verification method to preHandle #6318
Conversation
Signed-off-by: Matt Hess <matt.hess@swirldslabs.com>
Signed-off-by: Matt Hess <matt.hess@swirldslabs.com>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## develop #6318 +/- ##
==============================================
+ Coverage 67.86% 91.00% +23.14%
+ Complexity 22170 17156 -5014
==============================================
Files 2005 1312 -693
Lines 135357 49321 -86036
Branches 7637 4949 -2688
==============================================
- Hits 91854 44883 -46971
+ Misses 42011 3490 -38521
+ Partials 1492 948 -544
☔ View full report in Codecov by Sentry. |
...n/java/com/hedera/node/app/service/token/impl/handlers/TokenRevokeKycFromAccountHandler.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.
One comment on naming, otherwise LGTM !
Signed-off-by: Matt Hess <matt.hess@swirldslabs.com>
9286bad
to
bfbd19f
Compare
Kudos, SonarCloud Quality Gate passed! |
No description provided.