Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(go_sec): add error checking in deferring in io.close() #6144

Merged
merged 3 commits into from
Feb 7, 2023
Merged

fix(go_sec): add error checking in deferring in io.close() #6144

merged 3 commits into from
Feb 7, 2023

Conversation

cxMiguelSilva
Copy link
Collaborator

@cxMiguelSilva cxMiguelSilva commented Feb 7, 2023 •
edited
Loading

Proposed Changes

I submit this contribution under the Apache-2.0 license.

@cxMiguelSilva cxMiguelSilva added the go Pull requests that update Go code label Feb 7, 2023
@cxMiguelSilva cxMiguelSilva self-assigned this Feb 7, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Feb 7, 2023

kics-logo

KICS version: v1.6.6

Category Results
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 1
Files parsed placeholder 1
Files failed to scan placeholder 0
Total executed queries placeholder 49
Queries failed to execute placeholder 0
Execution time placeholder 1

@cxMiguelSilva cxMiguelSilva merged commit d919f86 into master Feb 7, 2023
@cxMiguelSilva cxMiguelSilva deleted the fix_go_sec branch February 7, 2023 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gabriel-cx gabriel-cx gabriel-cx approved these changes

@joaoReigota1 joaoReigota1 Awaiting requested review from joaoReigota1

Assignees

@cxMiguelSilva cxMiguelSilva

Labels
go Pull requests that update Go code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cxMiguelSilva @gabriel-cx