feat: add SetBeforeSendFeedback to process feedback before sending #4092

src/Sentry/Internal/MemoryMonitor.cs

@@ -100,7 +100,37 @@
             return;
         }
 
+
+        const long maxDumpSizeInBytes = 20 * 1024 * 1024; 
+
+        var fileInfo = new FileInfo(dumpFile);
+        if (fileInfo.Length > maxDumpSizeInBytes)
+        {
+            _options.LogWarning($"Memory dump file is too large ({fileInfo.Length} bytes). and will not be attached to the Sentry event.");
+
+            try
+            {
+                File.Delete(dumpFile);
+            }
+            catch (Exception ex)
+            {
+                _options.LogError($"Error deleting memory dump file: {ex.Message}");
+            }
+
+            return;
+        }
+
+
         _onDumpCollected(dumpFile);
+
+        try
+        {
+            File.Delete(dumpFile);
+        }
+        catch (Exception ex)
+        {
+            _options.LogError($"Error deleting memory dump file after sending to Sentry: {ex.Message}");
+        }
     }
 
     /// <summary>

src/Sentry/SentryClient.cs

@@ -15,6 +15,7 @@
 /// <inheritdoc cref="IDisposable" />
 public class SentryClient : ISentryClient, IDisposable
 {
+    private readonly ITransport _transport;
     private readonly SentryOptions _options;
     private readonly ISessionManager _sessionManager;
     private readonly RandomValuesFactory _randomValuesFactory;
@@ -37,7 +38,7 @@
     public SentryClient(SentryOptions options)
         : this(options, null, null, null) { }
 
     internal SentryClient(
         SentryOptions options,
         IBackgroundWorker? worker = null,
         RandomValuesFactory? randomValuesFactory = null,
@@ -84,9 +85,19 @@
     /// <inheritdoc />
     public void CaptureFeedback(SentryFeedback feedback, Scope? scope = null, SentryHint? hint = null)
     {
+
+        var processedFeedback = _options?.BeforeSendFeedbackInternal?.Invoke(feedback, hint);
+        if (processedFeedback is null)
+        {
+            _options?.DiagnosticLogger?.LogDebug("BeforeSendFeedback returned null, dropping feedback.");
+            return;
+        }
+
+        _transport.EnqueueFeedback(processedFeedback, hint);
+
         if (string.IsNullOrEmpty(feedback.Message))
         {
             _options.LogWarning("Feedback dropped due to empty message.");
             return;
         }
 
@@ -94,7 +105,7 @@
         hint ??= new SentryHint();
         hint.AddAttachmentsFromScope(scope);
 
         _options.LogInfo("Capturing feedback: '{0}'.", feedback.Message);
 
         var evt = new SentryEvent { Level = SentryLevel.Info };
         evt.Contexts.Feedback = feedback;

src/Sentry/SentryOptions.cs

@@ -442,6 +442,26 @@ internal bool IsSentryRequest(Uri? requestUri)
 
     internal Func<SentryEvent, SentryHint, SentryEvent?>? BeforeSendInternal => _beforeSend;
 
+    private Func<SentryFeedback, SentryHint?, SentryFeedback?>? _beforeSendFeedback;
+    internal Func<SentryFeedback, SentryHint?, SentryFeedback?>? BeforeSendFeedbackInternal => _beforeSendFeedback;
+
+
+    /// <summary>
+    /// Configures a callback to invoke before sending feedback to Sentry.
+    /// </summary>
+    public void SetBeforeSendFeedback(Func<SentryFeedback, SentryHint?, SentryFeedback?> beforeSendFeedback)
+    {
+        _beforeSendFeedback = beforeSendFeedback;
+    }
+
+    /// <summary>
+    /// Configures a callback to invoke before sending feedback to Sentry.
+    /// </summary>
+    public void SetBeforeSendFeedback(Func<SentryFeedback, SentryFeedback?> beforeSendFeedback)
+    {
+        _beforeSendFeedback = (feedback, _) => beforeSendFeedback(feedback);
+    }
+
     /// <summary>
     /// Configures a callback function to be invoked before sending an event to Sentry
     /// </summary>