Skip to content
Open
Show file tree
Hide file tree
Changes from 14 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,12 @@

## Unreleased

### Features

- Fail the build when OpenTelemetry is downgraded below the version the Sentry OpenTelemetry integration requires ([#1350](https://github.com/getsentry/sentry-android-gradle-plugin/pull/1350))
- The `sentry-opentelemetry-*` artifacts are built against specific OpenTelemetry versions. When another dependency management mechanism (most commonly Spring Boot's `io.spring.dependency-management`) forces OpenTelemetry below the version Sentry's integration requires, running against those downgraded versions can cause `ClassNotFoundException` / `NoSuchMethodError` at runtime. The new `verifySentryOpenTelemetryVersions` task detects this downgrade and fails the build early with guidance on how to fix it.
- You may disable this check by setting `sentry.autoInstallation.verifyOpenTelemetryVersions = false`

### Fixes

- Use Sentry BOM versions for auto-installed SDK dependencies ([#1349](https://github.com/getsentry/sentry-android-gradle-plugin/pull/1349))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,12 @@ open class AutoInstallExtension @Inject constructor(objects: ObjectFactory) {
*/
val sentryVersion: Property<String> =
objects.property(String::class.java).convention(SENTRY_SDK_VERSION)

/**
* Whether to verify that the OpenTelemetry versions resolved on the runtime classpath satisfy
* what the Sentry OpenTelemetry integration requires, failing the build if any were downgraded.
* Defaults to `true`.
*/
val verifyOpenTelemetryVersions: Property<Boolean> =

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this isn't tied to auto install, the option shouldn't live under auto install.

objects.property(Boolean::class.java).convention(true)
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,105 @@
package io.sentry.android.gradle.tasks

import io.sentry.android.gradle.extensions.SentryPluginExtension
import io.sentry.android.gradle.telemetry.SentryTelemetryService
import io.sentry.android.gradle.telemetry.withSentryTelemetry
import org.gradle.api.DefaultTask
import org.gradle.api.GradleException
import org.gradle.api.Project
import org.gradle.api.artifacts.result.ResolvedComponentResult
import org.gradle.api.provider.Property
import org.gradle.api.provider.Provider
import org.gradle.api.tasks.Input
import org.gradle.api.tasks.Internal
import org.gradle.api.tasks.TaskAction
import org.gradle.api.tasks.TaskProvider
import org.gradle.work.DisableCachingByDefault

/**
* Fails the build when the OpenTelemetry versions resolved on the runtime classpath were downgraded
* below what the Sentry OpenTelemetry integration requires.
*
* Sentry's `sentry-opentelemetry-*` artifacts declare the exact OpenTelemetry versions they were
* built and tested against. When another dependency management mechanism (most commonly
* `io.spring.dependency-management` / the Spring Boot BOM) forces a lower OpenTelemetry version,
* running against those downgraded versions can cause `ClassNotFoundException` /
* `NoSuchMethodError` at runtime. We detect that downgrade here and fail fast with actionable
* guidance instead of letting it blow up at runtime.
*/
@DisableCachingByDefault(because = "Only validates resolved dependency versions and has no outputs")
abstract class SentryOpenTelemetryVersionCheckTask : DefaultTask() {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This task is missing either a @DisableCachingByDefault or a @CacheableTask annotation. Check the build failure for details.

In this case since we have no outputs it should probably be @DisableCachingByDefault. Otherwise we can write a simple text file in a task specific output directory and make this task cacheable.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Skipping the output file for now. We can always add that later if needed.


@get:Internal abstract val rootComponent: Property<ResolvedComponentResult>

@get:Input abstract val docsUrl: Property<String>

@get:Input abstract val springDependencyManagementApplied: Property<Boolean>

@get:Input abstract val hasSentryOpenTelemetryDependency: Property<Boolean>

@get:Input abstract val verifyEnabled: Property<Boolean>

init {
description =
"Checks that the resolved OpenTelemetry versions are compatible with the Sentry " +
"OpenTelemetry integration"
group = "verification"
// Reference task-owned properties only (not the extension/project) so the task stays
// configuration-cache compatible.
@Suppress("LeakingThis")
onlyIf { verifyEnabled.get() && hasSentryOpenTelemetryDependency.get() }
}

@TaskAction
fun action() {
val downgrades = SentryOpenTelemetryVersionChecker.collectDowngrades(rootComponent.get())
if (downgrades.isNotEmpty()) {
throw GradleException(
SentryOpenTelemetryVersionChecker.buildMessage(
downgrades,
docsUrl.get(),
springDependencyManagementApplied.get(),
)
)
}
}

companion object {
private const val SPRING_DEPENDENCY_MANAGEMENT_PLUGIN_ID = "io.spring.dependency-management"

fun register(
project: Project,
extension: SentryPluginExtension,
sentryTelemetryProvider: Provider<SentryTelemetryService>,
configurationName: String,
docsUrl: String,
): TaskProvider<SentryOpenTelemetryVersionCheckTask> {
return project.tasks.register(
"verifySentryOpenTelemetryVersions",
SentryOpenTelemetryVersionCheckTask::class.java,
) { task ->
val configuration = project.configurations.getByName(configurationName)
task.rootComponent.set(configuration.incoming.resolutionResult.rootComponent)
task.docsUrl.set(docsUrl)
// Resolved lazily (at end of configuration) so it reflects the final plugin set regardless
// of the order plugins are applied in the consumer's build.
task.springDependencyManagementApplied.set(
project.provider {
project.pluginManager.hasPlugin(SPRING_DEPENDENCY_MANAGEMENT_PLUGIN_ID)
}
)
// Cheap declared-dependency check (no graph resolution) so we skip entirely for the common
// case of projects that don't use Sentry OpenTelemetry at all.
task.hasSentryOpenTelemetryDependency.set(
project.provider {
configuration.allDependencies.any {
SentryOpenTelemetryVersionChecker.isSentryOpenTelemetryArtifact(it.group, it.name)
}
}
Comment thread
cursor[bot] marked this conversation as resolved.
)
task.verifyEnabled.set(extension.autoInstallation.verifyOpenTelemetryVersions)
task.withSentryTelemetry(extension, sentryTelemetryProvider)
}
}
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,162 @@
package io.sentry.android.gradle.tasks

import io.sentry.android.gradle.util.SemVer
import org.gradle.api.artifacts.ModuleVersionIdentifier
import org.gradle.api.artifacts.component.ModuleComponentSelector
import org.gradle.api.artifacts.result.ComponentSelectionCause
import org.gradle.api.artifacts.result.ResolvedComponentResult
import org.gradle.api.artifacts.result.ResolvedDependencyResult

internal object SentryOpenTelemetryVersionChecker {
private const val SENTRY_GROUP = "io.sentry"
private const val SENTRY_OPENTELEMETRY_ARTIFACT_PREFIX = "sentry-opentelemetry-"
private const val OTEL_GROUP = "io.opentelemetry"

data class VersionDowngrade(
val module: String,
val requested: String,
val resolved: String,
val requestedBy: String,
val sentryBomVersion: String,
val reason: String?,
)

fun collectDowngrades(root: ResolvedComponentResult): List<VersionDowngrade> {
val downgrades = linkedMapOf<String, VersionDowngrade>()
val visited = mutableSetOf<Any>()
val stack = ArrayDeque<ResolvedComponentResult>()
stack.addLast(root)
while (stack.isNotEmpty()) {
val component = stack.removeLast()
if (!visited.add(component.id)) {
continue
}
val sentryOpenTelemetryArtifact = component.sentryOpenTelemetryArtifact()
for (dependency in component.dependencies) {
if (dependency !is ResolvedDependencyResult) {
continue
}
if (sentryOpenTelemetryArtifact != null) {
val requested = dependency.requested
if (requested is ModuleComponentSelector && requested.isOpenTelemetry()) {
val resolvedVersion = dependency.selected.moduleVersion?.version
if (resolvedVersion != null && isDowngrade(requested.version, resolvedVersion)) {
val module = "${requested.group}:${requested.module}"
downgrades.putIfAbsent(
module,
VersionDowngrade(
module = module,
requested = requested.version,
resolved = resolvedVersion,
requestedBy = sentryOpenTelemetryArtifact.toString(),
sentryBomVersion = sentryOpenTelemetryArtifact.version,
reason = dependency.selected.downgradeReason(),
),
)
}
}
}
stack.addLast(dependency.selected)
}
}
return downgrades.values.toList()
}

fun isSentryOpenTelemetryArtifact(group: String?, name: String): Boolean =
group == SENTRY_GROUP && name.startsWith(SENTRY_OPENTELEMETRY_ARTIFACT_PREFIX)

fun buildMessage(
downgrades: List<VersionDowngrade>,
docsUrl: String,
springDependencyManagementApplied: Boolean,
): String {
val sentryBomVersion = downgrades.first().sentryBomVersion
val details =
downgrades.joinToString(separator = "\n") { downgrade ->
buildString {
append(" - ${downgrade.module}: Sentry requires ${downgrade.requested} ")
append("but ${downgrade.resolved} was resolved")
append("\n Requested by: ${downgrade.requestedBy}")
downgrade.reason?.let { append("\n Gradle selection reason: $it") }
}
}
val fix =
if (springDependencyManagementApplied) {
"""
|To fix this, import the Sentry OpenTelemetry BOM through io.spring.dependency-management so
|its versions win dependency resolution:
|
| dependencyManagement {
| imports {
| mavenBom("io.sentry:sentry-opentelemetry-bom:$sentryBomVersion")
| }
| }
"""
.trimMargin()
} else {
"""
|To fix this, add the Sentry OpenTelemetry BOM as a platform dependency so its versions win
|dependency resolution:
|
| dependencies {
| implementation(platform("io.sentry:sentry-opentelemetry-bom:$sentryBomVersion"))
| }
"""
.trimMargin()
}
return """
|Sentry detected that OpenTelemetry was downgraded below the version its integration requires.
|
|The Sentry OpenTelemetry integration was built against specific OpenTelemetry versions,
|but the following were downgraded by another dependency management mechanism:
|
|$details
|
|Running with these downgraded OpenTelemetry versions can cause
|ClassNotFoundException / NoSuchMethodError at runtime.
|
|$fix
|
|See $docsUrl for details.
|
|You can disable this check with:
|
| sentry {
| autoInstallation.verifyOpenTelemetryVersions = false
| }
"""
.trimMargin()
}

private fun ResolvedComponentResult.sentryOpenTelemetryArtifact(): ModuleVersionIdentifier? {
val moduleVersion = moduleVersion ?: return null
if (isSentryOpenTelemetryArtifact(moduleVersion.group, moduleVersion.name)) {
return moduleVersion
}
return null
}

private fun ResolvedComponentResult.downgradeReason(): String? =
selectionReason.descriptions
.lastOrNull {
it.cause == ComponentSelectionCause.CONSTRAINT ||
it.cause == ComponentSelectionCause.FORCED ||
it.cause == ComponentSelectionCause.CONFLICT_RESOLUTION ||
it.cause == ComponentSelectionCause.SELECTED_BY_RULE
}
?.description

private fun ModuleComponentSelector.isOpenTelemetry(): Boolean =
group == OTEL_GROUP || group.startsWith("$OTEL_GROUP.")

private fun isDowngrade(requested: String, resolved: String): Boolean {
if (requested.isBlank() || requested == resolved) {
return false
}
return try {
SemVer.parse(resolved) < SemVer.parse(requested)
} catch (e: IllegalArgumentException) {
false
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -315,6 +315,10 @@ abstract class SentryTelemetryService : BuildService<None>, BuildOperationListen
"autoInstallation_sentryVersion",
extension.autoInstallation.sentryVersion.get().toString(),
)
tags.put(
"autoInstallation_verifyOpenTelemetryVersions",
extension.autoInstallation.verifyOpenTelemetryVersions.get().toString(),
)
tags.put("includeDependenciesReport", extension.includeDependenciesReport.get().toString())
tags.put("includeSourceContext", extension.includeSourceContext.get().toString())
tags.put(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import io.sentry.android.gradle.extensions.SentryPluginExtension
import io.sentry.android.gradle.sourcecontext.OutputPaths
import io.sentry.android.gradle.sourcecontext.SourceContext
import io.sentry.android.gradle.tasks.SentryGenerateDebugMetaPropertiesTask
import io.sentry.android.gradle.tasks.SentryOpenTelemetryVersionCheckTask
import io.sentry.android.gradle.tasks.dependencies.SentryExternalDependenciesReportTaskV2
import io.sentry.android.gradle.telemetry.SentryTelemetryService
import io.sentry.android.gradle.util.SentryPluginUtils
Expand Down Expand Up @@ -130,6 +131,23 @@ constructor(private val buildEvents: BuildEventListenerRegistryInternal) : Plugi
}

project.installDependencies(extension, false)

val verifyOpenTelemetryVersionsTask =
SentryOpenTelemetryVersionCheckTask.register(
project = project,
extension = extension,
sentryTelemetryProvider = sentryTelemetryProvider,
configurationName = "runtimeClasspath",
docsUrl = OPENTELEMETRY_VERSION_MISMATCH_DOCS_URL,
)
// Gate compilation so any build that compiles, packages, runs or tests (assemble, bootJar,
// bootRun, check, ...) fails fast on an OpenTelemetry version mismatch.
project.tasks.named("classes").configure { it.dependsOn(verifyOpenTelemetryVersionsTask) }

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verify hook skips test builds

High Severity

The OpenTelemetry verify task is attached only as a dependency of classes, but the Java plugin’s test task does not depend on classes (it reaches main compilation via compileTestJavacompileJava). Typical ./gradlew test or ./gradlew check runs can therefore compile and execute tests without ever running verifySentryOpenTelemetryVersions, so OTel downgrades may not fail the build despite the stated goal.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e4f8f9b. Configure here.

}
}

companion object {
internal const val OPENTELEMETRY_VERSION_MISMATCH_DOCS_URL =
"https://docs.sentry.io/platforms/java/opentelemetry/troubleshooting/"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO docs don't exist yet

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

}
}
Loading
Loading