fix: only have 1 telemetry handler

Author: davhdavh

Catglobe.CgScript.Common/CgScriptTelemetry.cs

@@ -10,7 +10,7 @@ public static class CgScriptTelemetry
    /// <summary>
    /// Name of source
    /// </summary>
-   public const string TelemetrySourceName = "Catglobe.CgScript";
+   internal      const string TelemetrySourceName = "Catglobe.CgScript";
    /// <summary>
    /// ActivitySource for CgScript
    /// </summary>

Catglobe.CgScript.Common/HostExtensions.cs

@@ -0,0 +1,16 @@
+using OpenTelemetry.Trace;
+
+namespace Catglobe.CgScript.Common;
+
+/// <summary>
+/// Setup methods
+/// </summary>
+public static class HostExtensions
+{
+   /// <summary>
+   /// Register the CgScript telemetry source
+   /// </summary>
+   public static TracerProviderBuilder AddCgScriptInstrumentation(this TracerProviderBuilder builder) => builder.AddSource(CgScriptTelemetry.TelemetrySourceName);
+
+}
+

Catglobe.CgScript.Deployment/HostExtensions.cs

@@ -51,10 +51,5 @@ private static IServiceCollection AddCommonCgScript(IServiceCollection services)
       return services;
    }
 
-   /// <summary>
-   /// Register the CgScript telemetry source
-   /// </summary>
-   public static TracerProviderBuilder AddCgScriptInstrumentation(this TracerProviderBuilder builder) => builder.AddSource(CgScriptTelemetry.TelemetrySourceName);
-
 }
 

Catglobe.CgScript.Runtime/HostExtensions.cs

@@ -48,10 +48,5 @@ private static IServiceCollection AddCommonCgScript(IServiceCollection services,
         .AddHttpMessageHandler<CgScriptAuthHandler>();
       return services;
    }
-
-   /// <summary>
-   /// Register the CgScript telemetry source
-   /// </summary>
-   public static TracerProviderBuilder AddCgScriptInstrumentation(this TracerProviderBuilder builder) => builder.AddSource(CgScriptTelemetry.TelemetrySourceName);
 }
 

README.md

@@ -382,7 +382,7 @@ using OpenTelemetry.Trace;
 
 var builder = WebApplication.CreateBuilder(args);
 
-builder.Services.AddOpenTelemetryTracing(tracerProviderBuilder =>
+builder.Services.AddOpenTelemetry().WithTracing(tracerProviderBuilder =>
 {
     tracerProviderBuilder
         .AddAspNetCoreInstrumentation()
@@ -430,3 +430,7 @@ See supported scopes in your Catglobe site `https://mysite.catglobe.com/.well-kn
 ## Can I use AOT compilation for my C# with this library?
 
 Yes
+
+## Can I make a request during authentication?
+
+Yes, e.g. in `OnTicketReceived`, you can set the `httpContext.Items["access_token"]` and that will be used to make the next request.