feat: Add support to send OTEL traces via OTLP

samples/Sentry.Samples.OpenTelemetry.AspNetCore/Program.cs

@@ -7,6 +7,13 @@
 
 var builder = WebApplication.CreateBuilder(args);
 
+// Read the Sentry DSN from the environment variable, if it's not already set in code.
+#if SENTRY_DSN_DEFINED_IN_ENV
+var dsn = Environment.GetEnvironmentVariable("SENTRY_DSN") ?? throw new InvalidOperationException("SENTRY_DSN environment variable is not set");
+#else
+var dsn = SamplesShared.Dsn;
+#endif
+
 // OpenTelemetry Configuration
 // See https://opentelemetry.io/docs/instrumentation/net/getting-started/
 builder.Services.AddOpenTelemetry()
@@ -20,8 +27,8 @@
             // The two lines below take care of configuring sources for ASP.NET Core and HttpClient
             .AddAspNetCoreInstrumentation()
             .AddHttpClientInstrumentation()
-            // Finally we configure OpenTelemetry to send traces to Sentry
-            .AddSentry()
+            // Finally, we configure OpenTelemetry to send traces to Sentry
+            .AddSentry(dsn)
     );
 
 builder.WebHost.UseSentry(options =>

samples/Sentry.Samples.OpenTelemetry.AspNetCore/Sentry.Samples.OpenTelemetry.AspNetCore.csproj

@@ -11,6 +11,7 @@
     <PackageReference Include="OpenTelemetry.Instrumentation.AspNetCore" Version="1.8.1" />
     <PackageReference Include="OpenTelemetry.Instrumentation.Http" Version="1.8.1" />
     <PackageReference Include="OpenTelemetry.Instrumentation.Runtime" Version="1.8.0" />
+    <PackageReference Include="OpenTelemetry.Exporter.OpenTelemetryProtocol" Version="1.8.1" />
   </ItemGroup>
 
   <PropertyGroup Condition="'$(Configuration)' == 'Release'">

src/Sentry.OpenTelemetry/Sentry.OpenTelemetry.csproj

@@ -17,7 +17,7 @@
 
   <ItemGroup>
     <!-- Version 1.6.0 is the minimum version that does not have trim warnings   -->
-    <PackageReference Include="OpenTelemetry" Version="1.6.0" />
+    <PackageReference Include="OpenTelemetry.Exporter.OpenTelemetryProtocol" Version="1.6.0" />
   </ItemGroup>
 
   <ItemGroup>

src/Sentry.OpenTelemetry/SentryOptionsExtensions.cs

@@ -41,8 +41,8 @@ public static void UseOpenTelemetry(
     /// <summary>
     /// <para>Configures Sentry to use OpenTelemetry for distributed tracing.</para>
     /// <para>
-    /// Note: if you are using this method to configure Sentry to work with OpenTelemetry you will also have to call
-    /// <see cref="TracerProviderBuilderExtensions.AddSentry"/> when building your <see cref="TracerProviderBuilder"/>
+    /// Note: if you are using this method to configure Sentry to work with OpenTelemetry, you will also have to call
+    /// TracerProviderBuilderExtensions.AddSentry when building your <see cref="TracerProviderBuilder"/>
     /// to ensure OpenTelemetry sends trace information to Sentry.
     /// </para>
     /// </summary>

src/Sentry.OpenTelemetry/TracerProviderBuilderExtensions.cs

@@ -1,6 +1,7 @@
 using Microsoft.Extensions.DependencyInjection;
 using OpenTelemetry;
 using OpenTelemetry.Context.Propagation;
+using OpenTelemetry.Exporter;
 using OpenTelemetry.Trace;
 using Sentry.Extensibility;
 
@@ -11,27 +12,74 @@ namespace Sentry.OpenTelemetry;
 /// </summary>
 public static class TracerProviderBuilderExtensions
 {
-    /// <summary>
-    /// Ensures OpenTelemetry trace information is sent to Sentry.
-    /// </summary>
-    /// <param name="tracerProviderBuilder"><see cref="TracerProviderBuilder"/>.</param>
-    /// <param name="defaultTextMapPropagator">
-    ///     <para>The default TextMapPropagator to be used by OpenTelemetry.</para>
-    ///     <para>
-    ///         If this parameter is not supplied, the <see cref="SentryPropagator"/> will be used, which propagates the
-    ///         baggage header as well as Sentry trace headers.
-    ///     </para>
-    ///     <para>
-    ///         The <see cref="SentryPropagator"/> is required for Sentry's OpenTelemetry integration to work but you
-    ///         could wrap this in a <see cref="CompositeTextMapPropagator"/> if you needed other propagators as well.
-    ///     </para>
-    /// </param>
-    /// <returns>The supplied <see cref="TracerProviderBuilder"/> for chaining.</returns>
-    public static TracerProviderBuilder AddSentry(this TracerProviderBuilder tracerProviderBuilder, TextMapPropagator? defaultTextMapPropagator = null)
+    extension(TracerProviderBuilder tracerProviderBuilder)
     {
-        defaultTextMapPropagator ??= new SentryPropagator();
-        Sdk.SetDefaultTextMapPropagator(defaultTextMapPropagator);
-        return tracerProviderBuilder.AddProcessor(ImplementationFactory);
+        /// <summary>
+        /// Ensures OpenTelemetry trace information is sent to Sentry.
+        /// </summary>
+        /// <param name="defaultTextMapPropagator">
+        ///     <para>The default TextMapPropagator to be used by OpenTelemetry.</para>
+        ///     <para>
+        ///         If this parameter is not supplied, the <see cref="SentryPropagator"/> will be used, which propagates the
+        ///         baggage header as well as Sentry trace headers.
+        ///     </para>
+        ///     <para>
+        ///         The <see cref="SentryPropagator"/> is required for Sentry's OpenTelemetry integration to work but you
+        ///         could wrap this in a <see cref="CompositeTextMapPropagator"/> if you needed other propagators as well.
+        ///     </para>
+        /// </param>
+        /// <returns>The supplied <see cref="TracerProviderBuilder"/> for chaining.</returns>
+        public TracerProviderBuilder AddSentry(TextMapPropagator? defaultTextMapPropagator = null)
+        {
+            defaultTextMapPropagator ??= new SentryPropagator();
+            Sdk.SetDefaultTextMapPropagator(defaultTextMapPropagator);
+            return tracerProviderBuilder.AddProcessor(ImplementationFactory);
+        }
+
+        /// <summary>
+        /// Ensures OpenTelemetry trace information is sent to the Sentry OTLP endpoint.
+        /// </summary>
+        /// <param name="dsnString">The DSN for your Sentry project</param>
+        /// <param name="defaultTextMapPropagator">
+        ///     <para>The default TextMapPropagator to be used by OpenTelemetry.</para>
+        ///     <para>
+        ///         If this parameter is not supplied, the <see cref="SentryPropagator"/> will be used, which propagates the
+        ///         baggage header as well as Sentry trace headers.
+        ///     </para>
+        ///     <para>
+        ///         The <see cref="SentryPropagator"/> is required for Sentry's OpenTelemetry integration to work but you
+        ///         could wrap this in a <see cref="CompositeTextMapPropagator"/> if you needed other propagators as well.
+        ///     </para>
+        /// </param>
+        /// <returns>The supplied <see cref="TracerProviderBuilder"/> for chaining.</returns>
+        public TracerProviderBuilder AddSentry(string dsnString, TextMapPropagator? defaultTextMapPropagator = null)
+        {
+            if (string.IsNullOrWhiteSpace(dsnString))
+            {
+                throw new ArgumentException("OTLP endpoint must be provided", nameof(dsnString));
+            }
+
+            defaultTextMapPropagator ??= new SentryPropagator();
+            Sdk.SetDefaultTextMapPropagator(defaultTextMapPropagator);
+
+            if (Dsn.TryParse(dsnString) is not { } dsn)
+            {
+                return tracerProviderBuilder;
+            }
+
+            tracerProviderBuilder.AddOtlpExporter(options =>
+            {
+                options.Endpoint = dsn.GetOtlpTracesEndpointUri();
+                options.Protocol = OtlpExportProtocol.HttpProtobuf;
+                options.HttpClientFactory = () =>
+                {
+                    var client = new HttpClient();
+                    client.DefaultRequestHeaders.Add("X-Sentry-Auth", $"sentry sentry_key={dsn.PublicKey}");
+                    return client;
+                };
+            });
+            return tracerProviderBuilder.AddProcessor(ImplementationFactory);
+        }
     }
 
     internal static BaseProcessor<Activity> ImplementationFactory(IServiceProvider services)

src/Sentry/Dsn.cs

@@ -58,6 +58,8 @@ private Dsn(
 
     public Uri GetEnvelopeEndpointUri() => new(ApiBaseUri, "envelope/");
 
+    public Uri GetOtlpTracesEndpointUri() => new(ApiBaseUri, "integration/otlp/v1/traces");
+
     public override string ToString() => Source;
 
     public static bool IsDisabled(string? dsn) =>