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
Remove duplicate error in switch case #7381
Conversation
Removed code returned Are these return values same? or is it appropriate to return |
Both seem to be returning the same response back to the caller, so it doesn't matter which one gets removed. |
c07fccc
to
84b4e54
Compare
Codecov Report
@@ Coverage Diff @@
## master #7381 +/- ##
==========================================
+ Coverage 48.06% 48.08% +0.01%
==========================================
Files 297 297
Lines 46809 46807 -2
==========================================
+ Hits 22499 22506 +7
+ Misses 22248 22241 -7
+ Partials 2062 2060 -2
Continue to review full report at Codecov.
|
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
Fixes: minio#7380. crypto.ErrInvalidCustomerKey was being handled twice in toAPIErrorCode()
Mint Automation
7381-84b4e54/mint-xl.sh.log:
|
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
Fixes: #7380. crypto.ErrInvalidCustomerKey was being handled twice
in toAPIErrorCode()
Description
Motivation and Context
Regression
NoHow Has This Been Tested?
code clean up - no effect on behavior
Types of changes
Checklist:
mint
PR # here: )