Bump SentryVersion to 5.0.0-alpha.1 (#207)

.github/workflows/build.yaml

@@ -33,14 +33,21 @@ jobs:
 
       - name: Setup Android SDK
         uses: android-actions/setup-android@07976c6290703d34c16d382cb36445f98bb43b1f # v3.2.0
+        with:
+          log-accepted-android-sdk-licenses: false
+
+      - name: Install .NET SDK
+        uses: actions/setup-dotnet@v4
+        with:
+          dotnet-version: 9.0.100
 
-      - name: Install Android workload (macOS)
+      - name: Install Workloads (macOS)
         if: matrix.os == 'macos-latest'
-        run: sudo dotnet workload install maui-android
+        run: sudo dotnet workload restore
 
-      - name: Install Android workload (Windows/Linux)
+      - name: Install Workloads (Windows/Linux)
         if: matrix.os != 'macos-latest'
-        run: dotnet workload install maui-android
+        run: dotnet workload restore
 
       - name: Run batch build script
         if: matrix.os == 'windows-latest'

global.json

@@ -0,0 +1,7 @@
+{
+  "sdk": {
+    "version": "9.0.100",
+    "rollForward": "latestMinor",
+    "allowPrerelease": false
+  }
+}

src/Directory.Packages.props

@@ -1,7 +1,7 @@
 <Project>
   <PropertyGroup>
     <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
-    <SentryVersion>4.12.0</SentryVersion>
+    <SentryVersion>5.0.0-alpha.1</SentryVersion>
   </PropertyGroup>
   <ItemGroup>
     <PackageVersion Include="Sentry.Extensions.Logging" Version="$(SentryVersion)" />

src/SymbolCollector.Server/BatchFinalizer.cs

@@ -35,23 +35,20 @@ public Task CloseBatch(
 public class SymsorterBatchFinalizer : IBatchFinalizer
 {
     private readonly SymbolServiceOptions _options;
-    private readonly IMetricsPublisher _metrics;
     private readonly ILogger<SymsorterBatchFinalizer> _logger;
     private readonly ISymbolGcsWriter _gcsWriter;
     private readonly BundleIdGenerator _bundleIdGenerator;
     private readonly IHub _hub;
     private readonly string _symsorterOutputPath;
 
     public SymsorterBatchFinalizer(
-        IMetricsPublisher metrics,
         IOptions<SymbolServiceOptions> options,
         ISymbolGcsWriter gcsWriter,
         BundleIdGenerator bundleIdGenerator,
         IHub hub,
         ILogger<SymsorterBatchFinalizer> logger)
     {
         _options = options?.Value ?? throw new ArgumentNullException(nameof(options));
-        _metrics = metrics;
         _logger = logger ?? throw new ArgumentNullException(nameof(logger));
         _gcsWriter = gcsWriter ?? throw new ArgumentNullException(nameof(gcsWriter));
         _bundleIdGenerator = bundleIdGenerator;
@@ -89,7 +86,6 @@ traceHeader is not null
                 ? _hub.StartTransaction("CloseBatch", "batch.close", traceHeader)
                 : _hub.StartTransaction("CloseBatch", "batch.close");
 
-        var handle = _metrics.BeginGcsBatchUpload();
         _ = Task.Run(async () =>
             {
                 // get logger factory and create a logger for symsorter
@@ -189,7 +185,6 @@ into fileGroup
                     {
                         _logger.LogError(e, "Failed attempting to delete symsorter directory.");
                     }
-                    handle.Dispose();
                 }
 
             }, gcsUploadCancellation)
@@ -292,4 +287,4 @@ private bool SortSymbols(string batchLocation, SymbolUploadBatch batch, string s
         _logger.LogInformation("Symsorter processed: {count}", match.Groups["count"].Value);
         return false;
     }
-}
+}

src/SymbolCollector.Server/Startup.cs

@@ -75,17 +75,6 @@ public void ConfigureServices(IServiceCollection services)
             .AddJsonOptions(options =>
                 options.JsonSerializerOptions.PropertyNamingPolicy = JsonNamingPolicy.CamelCase);
 
-        services.AddSingleton<ISymbolServiceMetrics, ProxyMetricsPublisher>();
-        services.AddSingleton<ISymbolControllerMetrics, ProxyMetricsPublisher>();
-        services.AddSingleton<IMetricsPublisher, ProxyMetricsPublisher>();
-
-        services.AddSingleton<StatsDMetricsPublisher>();
-        services.AddSingleton<SentryMetricsPublisher>();
-
-        services.AddSingleton<ProxyMetricsPublisher>(c => new ProxyMetricsPublisher(
-            c.GetRequiredService<StatsDMetricsPublisher>(),
-            c.GetRequiredService<SentryMetricsPublisher>()));
-
         services.AddOptions<StatsDOptions>()
             .Configure<IConfiguration>((o, c) => c.Bind("StatsD", o))
             .Validate(o => !string.IsNullOrWhiteSpace(o.Host), "StatD host is required.");
@@ -200,22 +189,18 @@ private string GetSymsorterPath()
     private class SymbolServiceEventProcessor : ISentryEventProcessor
     {
         private readonly IWebHostEnvironment _environment;
-        private readonly IMetricsPublisher _metrics;
         private readonly SymbolServiceOptions _options;
         private readonly string _cores = Environment.ProcessorCount.ToString();
         public SymbolServiceEventProcessor(
             IWebHostEnvironment environment,
-            IMetricsPublisher metrics,
             IOptions<SymbolServiceOptions> options)
         {
             _environment = environment;
-            _metrics = metrics;
             _options = options.Value;
         }
 
         public SentryEvent? Process(SentryEvent @event)
         {
-            _metrics.SentryEventProcessed();
             @event.SetTag("server-endpoint", _options.BaseAddress ?? "?");
             @event.Contexts["SymbolServiceOptions"] = _options;