Skip to content

Commit

Permalink
Merge pull request #7337 from TheThingsNetwork/fix/dev-nonces-reuse-t…
Browse files Browse the repository at this point in the history
…oo-small

Fix reversed Join Server dev nonce metrics
  • Loading branch information
ryaplots authored Oct 15, 2024
2 parents b98a593 + 12629bd commit 4203826
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ For details about compatibility between different releases, see the **Commitment
### Fixed

- Potential leak of end devices of other (owned) applications in the top end devices panel in the application overview of the Console.
- Fix reversed Join Server dev nonce metrics.

### Security

Expand Down
4 changes: 2 additions & 2 deletions pkg/joinserver/observability.go
Original file line number Diff line number Diff line change
Expand Up @@ -130,9 +130,9 @@ func registerRejectJoin(ctx context.Context, req *ttnpb.JoinRequest, err error)
}

func registerDevNonceReuse(ctx context.Context, msg *ttnpb.JoinRequest) {
jsMetrics.devNonce.tooSmall.WithLabelValues(ctx, ttnpb.MACVersion_name[int32(msg.SelectedMacVersion)]).Inc()
jsMetrics.devNonce.reuse.WithLabelValues(ctx, ttnpb.MACVersion_name[int32(msg.SelectedMacVersion)]).Inc()
}

func registerDevNonceTooSmall(ctx context.Context, msg *ttnpb.JoinRequest) {
jsMetrics.devNonce.reuse.WithLabelValues(ctx, ttnpb.MACVersion_name[int32(msg.SelectedMacVersion)]).Inc()
jsMetrics.devNonce.tooSmall.WithLabelValues(ctx, ttnpb.MACVersion_name[int32(msg.SelectedMacVersion)]).Inc()
}

0 comments on commit 4203826

Please sign in to comment.