Hubs/Scopes Merge 31 - Fix EventProcessor ordering on Scopes (#3360)

adinauer · web-flow · commit d45c72148e9b · 2024-05-02T14:04:26.000+02:00
* replace hub with scopes

* Add Scopes

* Introduce `IScopes` interface.

* Replace `IHub` with `IScopes` in core

* Replace `IHub` with `IScopes` in android core

* Replace `IHub` with `IScopes` in android integrations

* Replace `IHub` with `IScopes` in apollo integrations

* Replace `IHub` with `IScopes` in okhttp integration

* Replace `IHub` with `IScopes` in graphql integration

* Replace `IHub` with `IScopes` in logging integrations

* Replace `IHub` with `IScopes` in more integrations

* Replace `IHub` with `IScopes` in OTel integration

* Replace `IHub` with `IScopes` in Spring 5 / Spring Boot 2 integrations

* Replace `IHub` with `IScopes` in Spring 6 / Spring Boot 3 integrations

* Replace `IHub` with `IScopes` in samples

* gitscopes -&gt; github

* Replace ThreadLocal with ScopesStorage

* Move client and throwable to span map to scope

* Add global scope

* use global scope in Scopes

* Implement pushScope popScope and withScope for Scopes

* Add pushIsolationScope; add fork methods to ISCope

* Use separate scopes for current, isolation and global scope; rename mainScopes to rootScopes

* Allow controlling which scope configureScope uses

* Combine scopes

* Use new API for CRONS integrations

* Add lifecycle helper

* Change spring integrations to use new API

* Use new API in servlet integrations

* Use new API for kotlin coroutines and wrapers for Supplier/Callable

* Discussion TODOs

* Fix breadcrumb ordering

* Mark TODOS with [HSM]

* Add getGlobalScope and forkedRootScopes to IScopes

* Fix EventProcessor ordering on scopes
diff --git a/sentry-android-core/api/sentry-android-core.api b/sentry-android-core/api/sentry-android-core.api
@@ -97,6 +97,7 @@ public final class io/sentry/android/core/AnrIntegrationFactory {
 
 public final class io/sentry/android/core/AnrV2EventProcessor : io/sentry/BackfillingEventProcessor {
 	public fun <init> (Landroid/content/Context;Lio/sentry/android/core/SentryAndroidOptions;Lio/sentry/android/core/BuildInfoProvider;)V
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 	public fun process (Lio/sentry/protocol/SentryTransaction;Lio/sentry/Hint;)Lio/sentry/protocol/SentryTransaction;
 }
@@ -236,6 +237,7 @@ public final class io/sentry/android/core/PhoneStateBreadcrumbsIntegration : io/
 
 public final class io/sentry/android/core/ScreenshotEventProcessor : io/sentry/EventProcessor {
 	public fun <init> (Lio/sentry/android/core/SentryAndroidOptions;Lio/sentry/android/core/BuildInfoProvider;)V
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 	public fun process (Lio/sentry/protocol/SentryTransaction;Lio/sentry/Hint;)Lio/sentry/protocol/SentryTransaction;
 }
@@ -386,6 +388,7 @@ public final class io/sentry/android/core/UserInteractionIntegration : android/a
 
 public final class io/sentry/android/core/ViewHierarchyEventProcessor : io/sentry/EventProcessor {
 	public fun <init> (Lio/sentry/android/core/SentryAndroidOptions;)V
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 	public fun process (Lio/sentry/protocol/SentryTransaction;Lio/sentry/Hint;)Lio/sentry/protocol/SentryTransaction;
 	public static fun snapshotViewHierarchy (Landroid/app/Activity;Lio/sentry/ILogger;)Lio/sentry/protocol/ViewHierarchy;
diff --git a/sentry-android-core/src/main/java/io/sentry/android/core/AnrV2EventProcessor.java b/sentry-android-core/src/main/java/io/sentry/android/core/AnrV2EventProcessor.java
@@ -429,6 +429,11 @@ private void setOptionsTags(final @NotNull SentryBaseEvent event) {
   }
   // endregion
 
+  @Override
+  public @Nullable Long getOrder() {
+    return 12000L;
+  }
+
   // region static values
   private void setStaticValues(final @NotNull SentryEvent event) {
     mergeUser(event);
diff --git a/sentry-android-core/src/main/java/io/sentry/android/core/DefaultAndroidEventProcessor.java b/sentry-android-core/src/main/java/io/sentry/android/core/DefaultAndroidEventProcessor.java
@@ -303,4 +303,9 @@ private void setSideLoadedInfo(final @NotNull SentryBaseEvent event) {
 
     return transaction;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 8000L;
+  }
 }
diff --git a/sentry-android-core/src/main/java/io/sentry/android/core/PerformanceAndroidEventProcessor.java b/sentry-android-core/src/main/java/io/sentry/android/core/PerformanceAndroidEventProcessor.java
@@ -254,4 +254,9 @@ private static SentrySpan timeSpanToSentrySpan(
         null,
         defaultSpanData);
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 9000L;
+  }
 }
diff --git a/sentry-android-core/src/main/java/io/sentry/android/core/ScreenshotEventProcessor.java b/sentry-android-core/src/main/java/io/sentry/android/core/ScreenshotEventProcessor.java
@@ -98,4 +98,9 @@ public ScreenshotEventProcessor(
     hint.set(ANDROID_ACTIVITY, activity);
     return event;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 10000L;
+  }
 }
diff --git a/sentry-android-core/src/main/java/io/sentry/android/core/ViewHierarchyEventProcessor.java b/sentry-android-core/src/main/java/io/sentry/android/core/ViewHierarchyEventProcessor.java
@@ -284,4 +284,9 @@ private static ViewHierarchyNode viewToNode(@NotNull final View view) {
 
     return node;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 11000L;
+  }
 }
diff --git a/sentry-opentelemetry/sentry-opentelemetry-core/api/sentry-opentelemetry-core.api b/sentry-opentelemetry/sentry-opentelemetry-core/api/sentry-opentelemetry-core.api
@@ -1,5 +1,6 @@
 public final class io/sentry/opentelemetry/OpenTelemetryLinkErrorEventProcessor : io/sentry/EventProcessor {
 	public fun <init> ()V
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 }
 
diff --git a/sentry-opentelemetry/sentry-opentelemetry-core/src/main/java/io/sentry/opentelemetry/OpenTelemetryLinkErrorEventProcessor.java b/sentry-opentelemetry/sentry-opentelemetry-core/src/main/java/io/sentry/opentelemetry/OpenTelemetryLinkErrorEventProcessor.java
@@ -99,4 +99,9 @@ public OpenTelemetryLinkErrorEventProcessor() {
 
     return event;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 6000L;
+  }
 }
diff --git a/sentry-servlet-jakarta/src/main/java/io/sentry/servlet/jakarta/SentryRequestHttpServletRequestProcessor.java b/sentry-servlet-jakarta/src/main/java/io/sentry/servlet/jakarta/SentryRequestHttpServletRequestProcessor.java
@@ -56,4 +56,9 @@ public SentryRequestHttpServletRequestProcessor(@NotNull HttpServletRequest http
   private static @Nullable String toString(final @Nullable Enumeration<String> enumeration) {
     return enumeration != null ? String.join(",", Collections.list(enumeration)) : null;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 4000L;
+  }
 }
diff --git a/sentry-servlet/src/main/java/io/sentry/servlet/SentryRequestHttpServletRequestProcessor.java b/sentry-servlet/src/main/java/io/sentry/servlet/SentryRequestHttpServletRequestProcessor.java
@@ -56,4 +56,9 @@ public SentryRequestHttpServletRequestProcessor(@NotNull HttpServletRequest http
   private static @Nullable String toString(final @Nullable Enumeration<String> enumeration) {
     return enumeration != null ? String.join(",", Collections.list(enumeration)) : null;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 4000L;
+  }
 }
diff --git a/sentry-spring-jakarta/api/sentry-spring-jakarta.api b/sentry-spring-jakarta/api/sentry-spring-jakarta.api
@@ -5,6 +5,7 @@ public final class io/sentry/spring/jakarta/BuildConfig {
 
 public final class io/sentry/spring/jakarta/ContextTagsEventProcessor : io/sentry/EventProcessor {
 	public fun <init> (Lio/sentry/SentryOptions;)V
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 }
 
@@ -43,6 +44,7 @@ public class io/sentry/spring/jakarta/SentryInitBeanPostProcessor : org/springfr
 
 public class io/sentry/spring/jakarta/SentryRequestHttpServletRequestProcessor : io/sentry/EventProcessor {
 	public fun <init> (Lio/sentry/spring/jakarta/tracing/TransactionNameProvider;Ljakarta/servlet/http/HttpServletRequest;)V
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 }
 
diff --git a/sentry-spring-jakarta/src/main/java/io/sentry/spring/jakarta/ContextTagsEventProcessor.java b/sentry-spring-jakarta/src/main/java/io/sentry/spring/jakarta/ContextTagsEventProcessor.java
@@ -38,4 +38,9 @@ public ContextTagsEventProcessor(final @NotNull SentryOptions options) {
     }
     return event;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 14000L;
+  }
 }
diff --git a/sentry-spring-jakarta/src/main/java/io/sentry/spring/jakarta/SentryRequestHttpServletRequestProcessor.java b/sentry-spring-jakarta/src/main/java/io/sentry/spring/jakarta/SentryRequestHttpServletRequestProcessor.java
@@ -8,6 +8,7 @@
 import io.sentry.util.Objects;
 import jakarta.servlet.http.HttpServletRequest;
 import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 
 /** Attaches transaction name from the HTTP request to {@link SentryEvent}. */
 @Open
@@ -30,4 +31,9 @@ public SentryRequestHttpServletRequestProcessor(
     }
     return event;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 5000L;
+  }
 }
diff --git a/sentry-spring-jakarta/src/main/java/io/sentry/spring/jakarta/SentrySpringFilter.java b/sentry-spring-jakarta/src/main/java/io/sentry/spring/jakarta/SentrySpringFilter.java
@@ -24,6 +24,7 @@
 import jakarta.servlet.http.HttpServletResponse;
 import java.io.IOException;
 import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 import org.springframework.http.MediaType;
 import org.springframework.util.MimeType;
 import org.springframework.web.filter.OncePerRequestFilter;
@@ -157,5 +158,10 @@ public RequestBodyExtractingEventProcessor(
       }
       return event;
     }
+
+    @Override
+    public @Nullable Long getOrder() {
+      return 3000L;
+    }
   }
 }
diff --git a/sentry-spring/api/sentry-spring.api b/sentry-spring/api/sentry-spring.api
@@ -5,6 +5,7 @@ public final class io/sentry/spring/BuildConfig {
 
 public final class io/sentry/spring/ContextTagsEventProcessor : io/sentry/EventProcessor {
 	public fun <init> (Lio/sentry/SentryOptions;)V
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 }
 
@@ -43,6 +44,7 @@ public class io/sentry/spring/SentryInitBeanPostProcessor : org/springframework/
 
 public class io/sentry/spring/SentryRequestHttpServletRequestProcessor : io/sentry/EventProcessor {
 	public fun <init> (Lio/sentry/spring/tracing/TransactionNameProvider;Ljavax/servlet/http/HttpServletRequest;)V
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 }
 
diff --git a/sentry-spring/src/main/java/io/sentry/spring/ContextTagsEventProcessor.java b/sentry-spring/src/main/java/io/sentry/spring/ContextTagsEventProcessor.java
@@ -38,4 +38,9 @@ public ContextTagsEventProcessor(final @NotNull SentryOptions options) {
     }
     return event;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 14000L;
+  }
 }
diff --git a/sentry-spring/src/main/java/io/sentry/spring/SentryRequestHttpServletRequestProcessor.java b/sentry-spring/src/main/java/io/sentry/spring/SentryRequestHttpServletRequestProcessor.java
@@ -8,6 +8,7 @@
 import io.sentry.util.Objects;
 import javax.servlet.http.HttpServletRequest;
 import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 
 /** Attaches transaction name from the HTTP request to {@link SentryEvent}. */
 @Open
@@ -30,4 +31,9 @@ public SentryRequestHttpServletRequestProcessor(
     }
     return event;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 5000L;
+  }
 }
diff --git a/sentry-spring/src/main/java/io/sentry/spring/SentrySpringFilter.java b/sentry-spring/src/main/java/io/sentry/spring/SentrySpringFilter.java
@@ -24,6 +24,7 @@
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 import org.springframework.http.MediaType;
 import org.springframework.util.MimeType;
 import org.springframework.web.filter.OncePerRequestFilter;
@@ -157,5 +158,10 @@ public RequestBodyExtractingEventProcessor(
       }
       return event;
     }
+
+    @Override
+    public @Nullable Long getOrder() {
+      return 3000L;
+    }
   }
 }
diff --git a/sentry/api/sentry.api b/sentry/api/sentry.api
@@ -255,6 +255,7 @@ public final class io/sentry/CombinedScopeView : io/sentry/IScope {
 	public fun getLastEventId ()Lio/sentry/protocol/SentryId;
 	public fun getLevel ()Lio/sentry/SentryLevel;
 	public fun getOptions ()Lio/sentry/SentryOptions;
+	public fun getOrderedEventProcessors ()Ljava/util/List;
 	public fun getPropagationContext ()Lio/sentry/PropagationContext;
 	public fun getRequest ()Lio/sentry/protocol/Request;
 	public fun getScreen ()Ljava/lang/String;
@@ -342,6 +343,7 @@ public final class io/sentry/DateUtils {
 
 public final class io/sentry/DeduplicateMultithreadedEventProcessor : io/sentry/EventProcessor {
 	public fun <init> (Lio/sentry/SentryOptions;)V
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 }
 
@@ -377,6 +379,7 @@ public final class io/sentry/DsnUtil {
 
 public final class io/sentry/DuplicateEventDetectionEventProcessor : io/sentry/EventProcessor {
 	public fun <init> (Lio/sentry/SentryOptions;)V
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 }
 
@@ -392,6 +395,7 @@ public final class io/sentry/EnvelopeSender : io/sentry/IEnvelopeSender {
 }
 
 public abstract interface class io/sentry/EventProcessor {
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 	public fun process (Lio/sentry/protocol/SentryTransaction;Lio/sentry/Hint;)Lio/sentry/protocol/SentryTransaction;
 }
@@ -788,6 +792,7 @@ public abstract interface class io/sentry/IScope {
 	public abstract fun getLastEventId ()Lio/sentry/protocol/SentryId;
 	public abstract fun getLevel ()Lio/sentry/SentryLevel;
 	public abstract fun getOptions ()Lio/sentry/SentryOptions;
+	public abstract fun getOrderedEventProcessors ()Ljava/util/List;
 	public abstract fun getPropagationContext ()Lio/sentry/PropagationContext;
 	public abstract fun getRequest ()Lio/sentry/protocol/Request;
 	public abstract fun getScreen ()Ljava/lang/String;
@@ -1161,6 +1166,7 @@ public abstract interface class io/sentry/JsonUnknown {
 public final class io/sentry/MainEventProcessor : io/sentry/EventProcessor, java/io/Closeable {
 	public fun <init> (Lio/sentry/SentryOptions;)V
 	public fun close ()V
+	public fun getOrder ()Ljava/lang/Long;
 	public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;
 	public fun process (Lio/sentry/protocol/SentryTransaction;Lio/sentry/Hint;)Lio/sentry/protocol/SentryTransaction;
 }
@@ -1448,6 +1454,7 @@ public final class io/sentry/NoOpScope : io/sentry/IScope {
 	public fun getLastEventId ()Lio/sentry/protocol/SentryId;
 	public fun getLevel ()Lio/sentry/SentryLevel;
 	public fun getOptions ()Lio/sentry/SentryOptions;
+	public fun getOrderedEventProcessors ()Ljava/util/List;
 	public fun getPropagationContext ()Lio/sentry/PropagationContext;
 	public fun getRequest ()Lio/sentry/protocol/Request;
 	public fun getScreen ()Ljava/lang/String;
@@ -1891,6 +1898,7 @@ public final class io/sentry/Scope : io/sentry/IScope {
 	public fun getLastEventId ()Lio/sentry/protocol/SentryId;
 	public fun getLevel ()Lio/sentry/SentryLevel;
 	public fun getOptions ()Lio/sentry/SentryOptions;
+	public fun getOrderedEventProcessors ()Ljava/util/List;
 	public fun getPropagationContext ()Lio/sentry/PropagationContext;
 	public fun getRequest ()Lio/sentry/protocol/Request;
 	public fun getScreen ()Ljava/lang/String;
@@ -1961,6 +1969,7 @@ public abstract class io/sentry/ScopeObserverAdapter : io/sentry/IScopeObserver
 }
 
 public final class io/sentry/ScopeType : java/lang/Enum {
+	public static final field COMBINED Lio/sentry/ScopeType;
 	public static final field CURRENT Lio/sentry/ScopeType;
 	public static final field GLOBAL Lio/sentry/ScopeType;
 	public static final field ISOLATION Lio/sentry/ScopeType;
@@ -3808,6 +3817,14 @@ public final class io/sentry/internal/debugmeta/ResourcesDebugMetaLoader : io/se
 	public fun loadDebugMeta ()Ljava/util/List;
 }
 
+public final class io/sentry/internal/eventprocessor/EventProcessorAndOrder : java/lang/Comparable {
+	public fun <init> (Lio/sentry/EventProcessor;Ljava/lang/Long;)V
+	public fun compareTo (Lio/sentry/internal/eventprocessor/EventProcessorAndOrder;)I
+	public synthetic fun compareTo (Ljava/lang/Object;)I
+	public fun getEventProcessor ()Lio/sentry/EventProcessor;
+	public fun getOrder ()Ljava/lang/Long;
+}
+
 public abstract interface class io/sentry/internal/gestures/GestureTargetLocator {
 	public abstract fun locate (Ljava/lang/Object;FFLio/sentry/internal/gestures/UiElement$Type;)Lio/sentry/internal/gestures/UiElement;
 }
@@ -5384,6 +5401,11 @@ public final class io/sentry/util/DebugMetaPropertiesApplier {
 	public static fun getProguardUuid (Ljava/util/Properties;)Ljava/lang/String;
 }
 
+public final class io/sentry/util/EventProcessorUtils {
+	public fun <init> ()V
+	public static fun unwrap (Ljava/util/List;)Ljava/util/List;
+}
+
 public final class io/sentry/util/ExceptionUtils {
 	public fun <init> ()V
 	public static fun findRootCause (Ljava/lang/Throwable;)Ljava/lang/Throwable;
diff --git a/sentry/src/main/java/io/sentry/CombinedScopeView.java b/sentry/src/main/java/io/sentry/CombinedScopeView.java
@@ -1,9 +1,11 @@
 package io.sentry;
 
+import io.sentry.internal.eventprocessor.EventProcessorAndOrder;
 import io.sentry.protocol.Contexts;
 import io.sentry.protocol.Request;
 import io.sentry.protocol.SentryId;
 import io.sentry.protocol.User;
+import io.sentry.util.EventProcessorUtils;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
@@ -341,15 +343,20 @@ public void clearAttachments() {
   }
 
   @Override
-  public @NotNull List<EventProcessor> getEventProcessors() {
-    // TODO [HSM] mechanism for ordering event processors
-    final @NotNull List<EventProcessor> allEventProcessors = new CopyOnWriteArrayList<>();
-    allEventProcessors.addAll(globalScope.getEventProcessors());
-    allEventProcessors.addAll(isolationScope.getEventProcessors());
-    allEventProcessors.addAll(scope.getEventProcessors());
+  public @NotNull List<EventProcessorAndOrder> getOrderedEventProcessors() {
+    final @NotNull List<EventProcessorAndOrder> allEventProcessors = new CopyOnWriteArrayList<>();
+    allEventProcessors.addAll(globalScope.getOrderedEventProcessors());
+    allEventProcessors.addAll(isolationScope.getOrderedEventProcessors());
+    allEventProcessors.addAll(scope.getOrderedEventProcessors());
+    Collections.sort(allEventProcessors);
     return allEventProcessors;
   }
 
+  @Override
+  public @NotNull List<EventProcessor> getEventProcessors() {
+    return EventProcessorUtils.unwrap(getOrderedEventProcessors());
+  }
+
   @Override
   public void addEventProcessor(@NotNull EventProcessor eventProcessor) {
     getDefaultWriteScope().addEventProcessor(eventProcessor);
diff --git a/sentry/src/main/java/io/sentry/DeduplicateMultithreadedEventProcessor.java b/sentry/src/main/java/io/sentry/DeduplicateMultithreadedEventProcessor.java
@@ -62,4 +62,9 @@ public DeduplicateMultithreadedEventProcessor(final @NotNull SentryOptions optio
     processedEvents.put(type, currentEventTid);
     return event;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 7000L;
+  }
 }
diff --git a/sentry/src/main/java/io/sentry/DuplicateEventDetectionEventProcessor.java b/sentry/src/main/java/io/sentry/DuplicateEventDetectionEventProcessor.java
@@ -62,4 +62,9 @@ private static <T> boolean containsAnyKey(
     }
     return causes;
   }
+
+  @Override
+  public @Nullable Long getOrder() {
+    return 1000L;
+  }
 }
diff --git a/sentry/src/main/java/io/sentry/EventProcessor.java b/sentry/src/main/java/io/sentry/EventProcessor.java
diff --git a/sentry/src/main/java/io/sentry/IScope.java b/sentry/src/main/java/io/sentry/IScope.java
diff --git a/sentry/src/main/java/io/sentry/MainEventProcessor.java b/sentry/src/main/java/io/sentry/MainEventProcessor.java
diff --git a/sentry/src/main/java/io/sentry/NoOpScope.java b/sentry/src/main/java/io/sentry/NoOpScope.java
diff --git a/sentry/src/main/java/io/sentry/Scope.java b/sentry/src/main/java/io/sentry/Scope.java
diff --git a/sentry/src/main/java/io/sentry/SentryClient.java b/sentry/src/main/java/io/sentry/SentryClient.java
diff --git a/sentry/src/main/java/io/sentry/SentryRuntimeEventProcessor.java b/sentry/src/main/java/io/sentry/SentryRuntimeEventProcessor.java
diff --git a/sentry/src/main/java/io/sentry/internal/eventprocessor/EventProcessorAndOrder.java b/sentry/src/main/java/io/sentry/internal/eventprocessor/EventProcessorAndOrder.java
diff --git a/sentry/src/main/java/io/sentry/util/EventProcessorUtils.java b/sentry/src/main/java/io/sentry/util/EventProcessorUtils.java
diff --git a/sentry/src/test/java/io/sentry/CombinedScopeViewTest.kt b/sentry/src/test/java/io/sentry/CombinedScopeViewTest.kt

Original file line number	Diff line number	Diff line change
`@@ -303,4 +303,9 @@ private void setSideLoadedInfo(final @NotNull SentryBaseEvent event) {`
`303`	`303`
`304`	`304`	`return transaction;`
`305`	`305`	`}`
	`306`	`+`
	`307`	`+ @Override`
	`308`	`+ public @Nullable Long getOrder() {`
	`309`	`+ return 8000L;`
	`310`	`+ }`
`306`	`311`	`}`
Original file line number	Diff line number	Diff line change
`@@ -254,4 +254,9 @@ private static SentrySpan timeSpanToSentrySpan(`
`254`	`254`	`null,`
`255`	`255`	`defaultSpanData);`
`256`	`256`	`}`
	`257`	`+`
	`258`	`+ @Override`
	`259`	`+ public @Nullable Long getOrder() {`
	`260`	`+ return 9000L;`
	`261`	`+ }`
`257`	`262`	`}`
Original file line number	Diff line number	Diff line change
`@@ -98,4 +98,9 @@ public ScreenshotEventProcessor(`
`98`	`98`	`hint.set(ANDROID_ACTIVITY, activity);`
`99`	`99`	`return event;`
`100`	`100`	`}`
	`101`	`+`
	`102`	`+ @Override`
	`103`	`+ public @Nullable Long getOrder() {`
	`104`	`+ return 10000L;`
	`105`	`+ }`
`101`	`106`	`}`
Original file line number	Diff line number	Diff line change
`@@ -284,4 +284,9 @@ private static ViewHierarchyNode viewToNode(@NotNull final View view) {`
`284`	`284`
`285`	`285`	`return node;`
`286`	`286`	`}`
	`287`	`+`
	`288`	`+ @Override`
	`289`	`+ public @Nullable Long getOrder() {`
	`290`	`+ return 11000L;`
	`291`	`+ }`
`287`	`292`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,6 @@`
`1`	`1`	`public final class io/sentry/opentelemetry/OpenTelemetryLinkErrorEventProcessor : io/sentry/EventProcessor {`
`2`	`2`	`public fun <init> ()V`
	`3`	`+ public fun getOrder ()Ljava/lang/Long;`
`3`	`4`	`public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;`
`4`	`5`	`}`
`5`	`6`
Original file line number	Diff line number	Diff line change
`@@ -99,4 +99,9 @@ public OpenTelemetryLinkErrorEventProcessor() {`
`99`	`99`
`100`	`100`	`return event;`
`101`	`101`	`}`
	`102`	`+`
	`103`	`+ @Override`
	`104`	`+ public @Nullable Long getOrder() {`
	`105`	`+ return 6000L;`
	`106`	`+ }`
`102`	`107`	`}`
Original file line number	Diff line number	Diff line change
`@@ -56,4 +56,9 @@ public SentryRequestHttpServletRequestProcessor(@NotNull HttpServletRequest http`
`56`	`56`	`private static @Nullable String toString(final @Nullable Enumeration<String> enumeration) {`
`57`	`57`	`return enumeration != null ? String.join(",", Collections.list(enumeration)) : null;`
`58`	`58`	`}`
	`59`	`+`
	`60`	`+ @Override`
	`61`	`+ public @Nullable Long getOrder() {`
	`62`	`+ return 4000L;`
	`63`	`+ }`
`59`	`64`	`}`
Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@ public final class io/sentry/spring/jakarta/BuildConfig {`
`5`	`5`
`6`	`6`	`public final class io/sentry/spring/jakarta/ContextTagsEventProcessor : io/sentry/EventProcessor {`
`7`	`7`	`public fun <init> (Lio/sentry/SentryOptions;)V`
	`8`	`+ public fun getOrder ()Ljava/lang/Long;`
`8`	`9`	`public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;`
`9`	`10`	`}`
`10`	`11`
`@@ -43,6 +44,7 @@ public class io/sentry/spring/jakarta/SentryInitBeanPostProcessor : org/springfr`
`43`	`44`
`44`	`45`	`public class io/sentry/spring/jakarta/SentryRequestHttpServletRequestProcessor : io/sentry/EventProcessor {`
`45`	`46`	`public fun <init> (Lio/sentry/spring/jakarta/tracing/TransactionNameProvider;Ljakarta/servlet/http/HttpServletRequest;)V`
	`47`	`+ public fun getOrder ()Ljava/lang/Long;`
`46`	`48`	`public fun process (Lio/sentry/SentryEvent;Lio/sentry/Hint;)Lio/sentry/SentryEvent;`
`47`	`49`	`}`
`48`	`50`
Original file line number	Diff line number	Diff line change
`@@ -38,4 +38,9 @@ public ContextTagsEventProcessor(final @NotNull SentryOptions options) {`
`38`	`38`	`}`
`39`	`39`	`return event;`
`40`	`40`	`}`
	`41`	`+`
	`42`	`+ @Override`
	`43`	`+ public @Nullable Long getOrder() {`
	`44`	`+ return 14000L;`
	`45`	`+ }`
`41`	`46`	`}`
Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,7 @@`
`8`	`8`	`import io.sentry.util.Objects;`
`9`	`9`	`import jakarta.servlet.http.HttpServletRequest;`
`10`	`10`	`import org.jetbrains.annotations.NotNull;`
	`11`	`+import org.jetbrains.annotations.Nullable;`
`11`	`12`
`12`	`13`	`/** Attaches transaction name from the HTTP request to {@link SentryEvent}. */`
`13`	`14`	`@Open`
`@@ -30,4 +31,9 @@ public SentryRequestHttpServletRequestProcessor(`
`30`	`31`	`}`
`31`	`32`	`return event;`
`32`	`33`	`}`
	`34`	`+`
	`35`	`+ @Override`
	`36`	`+ public @Nullable Long getOrder() {`
	`37`	`+ return 5000L;`
	`38`	`+ }`
`33`	`39`	`}`