-
Notifications
You must be signed in to change notification settings - Fork 669
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
NOISSUE-Update invalid secret error handling #1856
NOISSUE-Update invalid secret error handling #1856
Conversation
f972baa
to
ba21556
Compare
Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com>
Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com>
ba21556
to
d243075
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.
Looks good, just a cosmetic fix of comment needed
Codecov Report
@@ Coverage Diff @@
## master #1856 +/- ##
==========================================
+ Coverage 64.12% 64.14% +0.02%
==========================================
Files 118 118
Lines 9642 9642
==========================================
+ Hits 6183 6185 +2
+ Misses 2801 2800 -1
+ Partials 658 657 -1
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com>
Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com>
Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com>
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
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
* update invalid secret error handling Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com> * update secret error naming Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com> * update secretError naming Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com> * fix comment Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com> * add empty space in comment Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com> --------- Signed-off-by: ianmuchyri <ianmuchiri8@gmail.com> Signed-off-by: SammyOina <sammyoina@gmail.com>
What does this do?
Updates error handling to handle invalid secret
Which issue(s) does this PR fix/relate to?
Put here
Resolves #XXX
to auto-close the issue that your PR fixes (if such)No issue
List any changes that modify/break current functionality
None
Have you included tests for your changes?
No
Did you document any new/modified functionality?
No
Notes