upsi. merging is hard

getsentry · Sep 26, 2024 · 976a4ea · 976a4ea
1 parent 0789910
commit 976a4ea
Showing 1 changed file with 13 additions and 2 deletions.
diff --git a/src/Sentry/Http/HttpTransportBase.cs b/src/Sentry/Http/HttpTransportBase.cs
@@ -452,9 +452,20 @@ private async Task HandleFailureAsync(HttpResponseMessage response, Envelope env
                 var destination = Path.Combine(destinationDirectory, "envelope_too_large",
                     (eventId ?? SentryId.Create()).ToString());
 
-                Directory.CreateDirectory(Path.GetDirectoryName(destination)!);
+                var createDirectoryResult = _options.FileSystem.CreateDirectory(Path.GetDirectoryName(destination)!);
+                if (createDirectoryResult is not FileOperationResult.Success)
+                {
+                    _options.DiagnosticLogger.LogError("Failed to create directory to store the envelope: {0}", createDirectoryResult);
+                    return;
+                }
+
+                var result = _options.FileSystem.CreateFileForWriting(destination, out var envelopeFile);
+                if (result is not FileOperationResult.Success)
+                {
+                    _options.DiagnosticLogger.LogError("Failed to create envelope file: {0}", result);
+                    return;
+                }
 
-                var envelopeFile = File.Create(destination);
 #if NETFRAMEWORK || NETSTANDARD2_0
                 using (envelopeFile)
 #else