Merge pull request #223 from peppy/no-sentry-for-debug

Don't use sentry when running in debug mode
ppy · Feb 28, 2024 · cc1d36d · cc1d36d
2 parents 7ae4bc3 + 71c2150
commit cc1d36d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/osu.Server.Spectator/Program.cs b/osu.Server.Spectator/Program.cs
@@ -40,15 +40,15 @@ private static IHostBuilder createHostBuilder(string[] args)
             return Host.CreateDefaultBuilder(args)
                        .ConfigureWebHostDefaults(webBuilder =>
                        {
+#if !DEBUG
                            webBuilder.UseSentry(o =>
                            {
                                o.AddExceptionFilterForType<HubException>();
                                o.TracesSampleRate = 0.01;
-#if !DEBUG
                                o.Dsn = "https://[email protected]/8";
-#endif
                                // TODO: set release name
                            });
+#endif
 
 #if DEBUG
                            webBuilder.UseStartup<StartupDevelopment>();

diff --git a/osu.Server.Spectator/Startup.cs b/osu.Server.Spectator/Startup.cs
@@ -67,7 +67,9 @@ public void ConfigureServices(IServiceCollection services)
 
                 logging.ClearProviders();
                 logging.AddConsole();
+#if !DEBUG
                 logging.AddSentry();
+#endif
 
                 // IdentityModelEventSource.ShowPII = true;
             });