Remove indirection for integrations and packages via VersionInfoHolder

Author: adinauer

sentry-opentelemetry/sentry-opentelemetry-agentcustomization/src/main/java/io/sentry/opentelemetry/SentryAutoConfigurationCustomizerProvider.java

@@ -46,15 +46,6 @@ public void customize(AutoConfigurationCustomizer autoConfiguration) {
           });
     }
 
-    if (versionInfoHolder != null) {
-      for (SentryPackage pkg : versionInfoHolder.getPackages()) {
-        SentryIntegrationPackageStorage.getInstance().addPackage(pkg.getName(), pkg.getVersion());
-      }
-      for (String integration : versionInfoHolder.getIntegrations()) {
-        SentryIntegrationPackageStorage.getInstance().addIntegration(integration);
-      }
-    }
-
     autoConfiguration
         .addTracerProviderCustomizer(this::configureSdkTracerProvider)
         .addPropertiesSupplier(this::getDefaultProperties);

sentry/src/main/java/io/sentry/internal/ManifestVersionReader.java

@@ -73,18 +73,16 @@ public void readManifestFiles() {
               final @Nullable String otelVersion =
                   mainAttributes.getValue("Sentry-Opentelemetry-Version-Name");
               if (otelVersion != null) {
-                versionInfo.packages.add(
-                    new SentryPackage("maven:io.opentelemetry:opentelemetry-sdk", otelVersion));
-                versionInfo.integrations.add("OpenTelemetry");
+                SentryIntegrationPackageStorage.getInstance().addPackage("maven:io.opentelemetry:opentelemetry-sdk", otelVersion);
+                SentryIntegrationPackageStorage.getInstance().addIntegration("OpenTelemetry");
               }
               final @Nullable String otelJavaagentVersion =
                   mainAttributes.getValue("Sentry-Opentelemetry-Javaagent-Version-Name");
               if (otelJavaagentVersion != null) {
-                versionInfo.packages.add(
-                    new SentryPackage(
+                SentryIntegrationPackageStorage.getInstance().addPackage(
                         "maven:io.opentelemetry.javaagent:opentelemetry-javaagent",
-                        otelJavaagentVersion));
-                versionInfo.integrations.add("OpenTelemetry-Agent");
+                        otelJavaagentVersion);
+                SentryIntegrationPackageStorage.getInstance().addIntegration("OpenTelemetry-Agent");
               }
               if (name.equals("sentry.java.opentelemetry.agentless")) {
                 SentryIntegrationPackageStorage.getInstance()
@@ -117,8 +115,6 @@ public void readManifestFiles() {
   public static final class VersionInfoHolder {
     private volatile @Nullable String sdkName;
     private volatile @Nullable String sdkVersion;
-    private List<SentryPackage> packages = new ArrayList<>();
-    private List<String> integrations = new ArrayList<>();
 
     public @Nullable String getSdkName() {
       return sdkName;
@@ -127,13 +123,5 @@ public static final class VersionInfoHolder {
     public @Nullable String getSdkVersion() {
       return sdkVersion;
     }
-
-    public List<SentryPackage> getPackages() {
-      return packages;
-    }
-
-    public List<String> getIntegrations() {
-      return integrations;
-    }
   }
 }