Skip to content

Fix gosec slice bounds warnings#213

Merged
JoTurk merged 1 commit intomasterfrom
fix-gosec-slice-bounds
Dec 20, 2025
Merged

Fix gosec slice bounds warnings#213
JoTurk merged 1 commit intomasterfrom
fix-gosec-slice-bounds

Conversation

@asayyah
Copy link
Contributor

@asayyah asayyah commented Dec 20, 2025

Add nolint directives for safe slice accesses where bounds are guaranteed by fixed-size allocation or loop conditions.

Add nolint directives for safe slice accesses where bounds
are guaranteed by fixed-size allocation or loop conditions.
@asayyah asayyah requested a review from JoTurk December 20, 2025 23:30
@codecov
Copy link

codecov bot commented Dec 20, 2025

Codecov Report

❌ Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 77.74%. Comparing base (99b388f) to head (d072ebe).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
receiver_estimated_maximum_bitrate.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #213      +/-   ##
==========================================
+ Coverage   77.07%   77.74%   +0.67%     
==========================================
  Files          22       22              
  Lines        2569     2022     -547     
==========================================
- Hits         1980     1572     -408     
+ Misses        493      354     -139     
  Partials       96       96              
Flag Coverage Δ
go 77.74% <88.88%> (+0.67%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@JoTurk JoTurk merged commit 603b7c3 into master Dec 20, 2025
19 checks passed
@JoTurk JoTurk deleted the fix-gosec-slice-bounds branch December 20, 2025 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants