fix: opentelemetry.enabled feature flag

Author: smarcet

app/Audit/AuditEventListener.php

@@ -74,6 +74,7 @@ private function getAuditStrategy($em): ?IAuditStrategy
         // Check if OTLP audit is enabled
         if (config('opentelemetry.enabled', false)) {
             try {
+                Log::debug("AuditEventListener::getAuditStrategy strategy AuditLogOtlpStrategy");
                 return App::make(AuditLogOtlpStrategy::class);
             } catch (\Exception $e) {
                 Log::warning('Failed to create OTLP audit strategy, falling back to database', [
@@ -83,6 +84,7 @@ private function getAuditStrategy($em): ?IAuditStrategy
         }
 
         // Use database strategy (either as default or fallback)
+        Log::debug("AuditEventListener::getAuditStrategy strategy AuditLogStrategy");
         return new AuditLogStrategy($em);
     }
 

app/Audit/AuditLogOtlpStrategy.php

@@ -30,6 +30,10 @@ public function __construct(IAuditLogFormatterFactory $formatterFactory)
         $this->formatterFactory = $formatterFactory;
         $this->enabled = config('opentelemetry.enabled', false);
         $this->elasticIndex = config('opentelemetry.logs.elasticsearch_index', 'logs-audit');
+        Log::debug("AuditLogOtlpStrategy::__construct", [
+            'enabled' => $this->enabled,
+            'elasticIndex' => $this->elasticIndex,
+        ]);
     }
 
     /**

app/Services/Apis/MailApi.php

@@ -12,7 +12,6 @@
  * limitations under the License.
  **/
 use GuzzleHttp\Client;
-use GuzzleHttp\Exception\ClientException;
 use GuzzleHttp\Exception\RequestException;
 use GuzzleHttp\HandlerStack;
 use GuzzleRetry\GuzzleRetryMiddleware;

config/opentelemetry.php

@@ -4,7 +4,7 @@
 use OpenTelemetry\SDK\Common\Configuration\Variables;
 
 return [
-    'enabled' => env('OTEL_SERVICE_ENABLED', false),
+    'enabled' => filter_var(env('OTEL_SERVICE_ENABLED', false), FILTER_VALIDATE_BOOLEAN),
     /**
      * Service name
      */