Skip to content

fix: goagent gosec issues #215

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 2 commits into from
Closed

fix: goagent gosec issues #215

wants to merge 2 commits into from

Conversation

tim-mwangi
Copy link
Collaborator

Description

Fix gosec issues

@codecov
Copy link

codecov bot commented Jul 24, 2023

Codecov Report

Attention: Patch coverage is 14.28571% with 6 lines in your changes missing coverage. Please review.

Project coverage is 58.52%. Comparing base (e89d845) to head (42efe3d).
Report is 41 commits behind head on main.

Files with missing lines Patch % Lines
instrumentation/opencensus/init.go 0.00% 3 Missing ⚠️
sdk/instrumentation/net/http/attributes.go 25.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #215      +/-   ##
==========================================
- Coverage   59.19%   58.52%   -0.67%     
==========================================
  Files          55       55              
  Lines        2235     2240       +5     
==========================================
- Hits         1323     1311      -12     
- Misses        854      869      +15     
- Partials       58       60       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tim-mwangi
Copy link
Collaborator Author

Closing this one

@tim-mwangi tim-mwangi closed this Jul 21, 2025
@tim-mwangi tim-mwangi deleted the gosec-fix branch July 21, 2025 15:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants