Skip to content

feat(android-distribution): Add build_configuration support and use snake_case for parsing responses #4800

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
runningcode merged 2 commits into from
Oct 13, 2025

Merge branch 'main' into no/add-build-configuration-support

f99fefe
Select commit
Loading
Failed to load commit list.
Merged

feat(android-distribution): Add build_configuration support and use snake_case for parsing responses #4800

Merge branch 'main' into no/add-build-configuration-support
f99fefe
Select commit
Loading
Failed to load commit list.
Cursor / Cursor Bugbot completed Oct 13, 2025 in 2m 4s

Bugbot Review

Bugbot Analysis Progress (2m 7s elapsed)

✅ Gathered PR context (2s)
✅ Analyzed code changes (1s)
✅ Completed bug detection — 1 potential bug found (1m 59s)
✅ Validation and filtering completed (1s)
✅ Posted analysis results — 1 bug reported (4s)
✅ Analysis completed successfully (0s)

Final Result: Bugbot completed review and found 1 potential issue

Request ID: serverGenReqId_0eac8410-e747-46a3-9c6b-785dc3cbcb66

Details

Bug: API Validation Error Naming Mismatch

The validation error messages for missing fields still reference camelCase names like buildVersion, but the parser now expects snake_case names such as build_version. This mismatch can make debugging API response issues confusing.

sentry-android-distribution/src/main/java/io/sentry/android/distribution/UpdateResponseParser.kt#L66-L74

sentry-java/sentry-android-distribution/src/main/java/io/sentry/android/distribution/UpdateResponseParser.kt

Lines 66 to 74 in f99fefe

}
if (buildVersion.isEmpty() || buildVersion == "null") {
missingFields.add("buildVersion")
}
if (downloadUrl.isEmpty() || downloadUrl == "null") {
missingFields.add("downloadUrl")
}
if (missingFields.isNotEmpty()) {

Fix in Cursor Fix in Web

View more details on Cursor