Skip to content

Commit

Permalink
Fix gosec lint errors
Browse files Browse the repository at this point in the history
Ignore gosec linting for integer conversion G115.
  • Loading branch information
acosta11 committed Sep 10, 2024
1 parent 1b8d052 commit cd733b8
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/router/app/router.go
Original file line number Diff line number Diff line change
Expand Up @@ -113,13 +113,13 @@ func (d *Router) Start() {
v1Buf := diodes.NewManyToOneEnvelope(d.c.IngressBufferSize, gendiodes.AlertFunc(func(missed int) {
log.Printf("Dropped %d envelopes (v1 buffer)", missed)

ingressDropped.Increment(uint64(missed))
ingressDropped.Increment(uint64(missed)) // nolint:gosec
}))

v2Buf := diodes.NewManyToOneEnvelopeV2(d.c.IngressBufferSize, gendiodes.AlertFunc(func(missed int) {
log.Printf("Dropped %d envelopes (v2 buffer)", missed)

ingressDropped.Increment(uint64(missed))
ingressDropped.Increment(uint64(missed)) // nolint:gosec
}))

// metric-documentation-v2: (loggregator.doppler.subscriptions) Number of
Expand Down
2 changes: 1 addition & 1 deletion src/router/internal/server/v1/doppler_server.go
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ func (m *DopplerServer) sendBatchData(req *plumbing.SubscriptionRequest, sender

// Alert logs dropped message counts to stderr.
func (m *DopplerServer) Alert(missed int) {
m.egressDropped.Increment(uint64(missed))
m.egressDropped.Increment(uint64(missed)) // nolint:gosec
}

func (m *DopplerServer) monitorContext(ctx context.Context, done *int64) {
Expand Down
2 changes: 1 addition & 1 deletion src/router/internal/server/v2/egress_server.go
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ func NewEgressServer(

// Alert logs dropped message counts to stderr.
func (s *EgressServer) Alert(missed int) {
s.droppedMetric.Increment(uint64(missed))
s.droppedMetric.Increment(uint64(missed)) // nolint:gosec
}

// Receiver implements loggregator_v2.EgressServer.
Expand Down

0 comments on commit cd733b8

Please sign in to comment.