Merge pull request #170 from RADAR-base/fix/open-telemetry

feat: add sentry open telemetry agent jar to build

Author: mpgxvii

README.md

@@ -215,7 +215,7 @@ To enable Sentry monitoring for the generic REST, Fitbit, or Oura source connect
 1. Set a `SENTRY_DSN` environment variable that points to the desired Sentry DSN.
 2. (Optional) Set the `SENTRY_LOG_LEVEL` environment variable to control the minimum log level of
    events sent to Sentry.
-   The default log level for Sentry is `WARN`. Possible values are `TRACE`, `DEBUG`, `INFO`, `WARN`,
+   The default log level for Sentry is `ERROR`. Possible values are `TRACE`, `DEBUG`, `INFO`, `WARN`,
    and `ERROR`.
 
 For further configuration of Sentry via environmental variables see [here](https://docs.sentry.io/platforms/java/configuration/#configuration-via-the-runtime-environment). For instance:

buildSrc/src/main/kotlin/Versions.kt

@@ -19,6 +19,7 @@ object Versions {
     const val log4j2 = "2.23.1"
     const val slf4j = "2.0.13"
     const val sentryLog4j = "1.7.30"
+    const val sentryOpenTelemetryAgent = "8.1.0"
 
     const val okhttp = "4.12.0"
 

kafka-connect-fitbit-source/build.gradle.kts

@@ -34,12 +34,5 @@ dependencies {
     compileOnly(platform("com.fasterxml.jackson:jackson-bom:${Versions.jackson}"))
     compileOnly("com.fasterxml.jackson.core:jackson-databind")
 
-    // Application monitoring
-    // This dependency is not used by the REST connector, but copied into the Docker image (Dockerfile)
-    compileOnly("io.sentry:sentry-log4j:${Versions.sentryLog4j}") {
-        // Exclude log4j with security vulnerability (safe version is provided by docker image).
-        exclude(group = "log4j", module = "log4j")
-    }
-
     testImplementation("org.apache.kafka:connect-api:${Versions.kafka}")
 }

kafka-connect-oura-source/build.gradle.kts

@@ -33,12 +33,5 @@ dependencies {
     compileOnly(platform("com.fasterxml.jackson:jackson-bom:${Versions.jackson}"))
     compileOnly("com.fasterxml.jackson.core:jackson-databind")
 
-    // Application monitoring
-    // This dependency is not used by the REST connector, but copied into the Docker image (Dockerfile)
-    compileOnly("io.sentry:sentry-log4j:${Versions.sentryLog4j}") {
-        // Exclude log4j with security vulnerability (safe version is provided by docker image).
-        exclude(group = "log4j", module = "log4j")
-    }
-
     testImplementation("org.apache.kafka:connect-api:${Versions.kafka}")
 }

kafka-connect-rest-source/build.gradle.kts

@@ -11,4 +11,12 @@ dependencies {
     testImplementation("com.github.tomakehurst:wiremock:${Versions.wiremock}")
 
     testImplementation("org.apache.kafka:connect-api:${Versions.kafka}")
+
+    // Application monitoring
+    // These dependencies are not used by the REST connector, but copied into the Docker image (Dockerfile)
+    runtimeOnly("io.sentry:sentry-log4j:${Versions.sentryLog4j}") {
+        // Exclude log4j with security vulnerability (safe version is provided by docker image).
+        exclude(group = "log4j", module = "log4j")
+    }
+    runtimeOnly("io.sentry:sentry-opentelemetry-agent:${Versions.sentryOpenTelemetryAgent}")
 }