SLVS-2801 SLVS-2764 Enable Sentry

[eray-felek-sonarsource]

No description provided.

[hashicorp-vault-sonar-prod]

SLVS-2801
SLVS-2764

[sonarqubecloud]

SonarQube reviewer guide

Important

We are currently testing different models for AI Summary.
Please give us your feedback by filling this form.

Model A:

Summary: Adds Sentry error monitoring with telemetry integration, upgrades StreamJsonRpc to v2.8.28, and reports RPC exceptions.

Review Focus:

• MonitoringService initialization/lifecycle management tied to telemetry opt-in/out
• Exception reporting logic in JsonRpcWrapper.CreateErrorDetails() that filters error types before reporting
• Error handling for VS version compatibility (multiple try-catch blocks swallowing exceptions)
• LocalRpcException handling with CommonErrorData extraction

Start review at: src/Integration/Telemetry/MonitoringService.cs. This is the core implementation that manages the Sentry SDK lifecycle, configures monitoring options, and determines which exceptions get reported - understanding this flow is critical before reviewing its integration points.

Model B:

Summary: Add Sentry error monitoring integration and upgrade StreamJsonRpc dependency.

Review Focus:

• The new MonitoringService class handles Sentry SDK initialization with telemetry-aware lifecycle management (Init/Close/Reinit)
• Error reporting is integrated into JsonRpc error handling to capture internal errors and invalid params
• StreamJsonRpc upgraded from 2.6.121 to 2.8.28, affecting multiple transitive dependencies
• Thread safety via locks in MonitoringService; ensure exception handling in opt-in/opt-out doesn't mask real issues

Start review at: src/Integration/Telemetry/MonitoringService.cs. This is the core new functionality that orchestrates Sentry initialization based on telemetry status, handles exception reporting with context enrichment, and manages the service lifecycle.

See all code changes, issues, and quality metrics in one place.

Quality Gate passed

Issues
0 New issues
4 Accepted issues
0 Dependency risks

Measures
0 Security Hotspots
90.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud