-
Notifications
You must be signed in to change notification settings - Fork 191
feat: adds custom scorecard certifier #2815
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
base: main
Are you sure you want to change the base?
feat: adds custom scorecard certifier #2815
Conversation
Kusari Analysis Results:
No pinned version dependency changes, code issues or exposed secrets detected! Note View full detailed analysis result for more information on the output and the checks that were run.
Found this helpful? Give it a 👍 or 👎 reaction! |
| req.Header.Set("Accept", "application/json") | ||
|
|
||
| resp, err := httpClient.Do(req) | ||
| defer func() { |
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.
Issue: Fix nil pointer dereference in defer statement
Recommended Code Changes:
resp, err := httpClient.Do(req)
if err != nil {
return nil, fmt.Errorf("scorecard request failed: %w", err)
}
defer func() {
_ = resp.Body.Close()
}()
|
Kusari PR Analysis rerun based on - 81bb024 performed at: 2025-11-24T03:05:26Z - link to updated analysis |
0b3c224 to
e0151f0
Compare
98202c8 to
845216e
Compare
Co-authored-by: Shreyas Pandya <pandyashreyas1@gmail.com> Co-authored-by: Abhisek Agrawal <abhishek.yours4@gmail.com> Co-authored-by: Anirudh Edpuganti <aniedpuganti@gmail.com> Co-authored-by: Gagan H R <hrgagan4@gmail.com> Signed-off-by: Gagan H R <hrgagan4@gmail.com>
845216e to
3f18d64
Compare
Fixes #2783
Description of the PR
PR Checklist
-sflag togit commit.make generatehas been runmake generatehas been runmake generatehas been runcollectsubprotobuf has been changed,make protohas been run