Skip to content

Respect UploadDevelopmentSymbols for iOS symbol upload - #2928

Open
bitsandfoxes wants to merge 2 commits into
test/skip-android-2021-throw-framefrom
fix/ios-upload-development-symbols
Open

bitsandfoxes wants to merge 2 commits into
test/skip-android-2021-throw-framefrom
fix/ios-upload-development-symbols

Conversation

@bitsandfoxes

Copy link
Copy Markdown
Contributor

On iOS, symbols were uploaded for development builds even with UploadDevelopmentSymbols off: the Xcode SENTRY_CLI_UPLOAD_SYMBOLS build setting was set from UploadSymbols alone. Android and the native platforms already skip development builds unless UploadDevelopmentSymbols is on, through SentryCliOptions.IsValid.

  • The upload and development-build checks at the start of IsValid are now their own internal method, SentryCliOptions.IsUploadEnabled, which IsValid calls. Behaviour on the other platforms is unchanged.
  • SentryXcodeProject.AddBuildPhaseSymbolUpload takes isDevelopmentBuild and sets SENTRY_CLI_UPLOAD_SYMBOLS from IsUploadEnabled. The iOS build post-processor passes EditorUserBuildSettings.development, the same flag the other platforms use.
  • iOS still doesn't require an auth token at export time, and the upload build phase is still always added; with the setting at NO, the phase skips the upload.

Test: AddBuildPhaseSymbolUpload_DevelopmentBuild_RespectsUploadDevelopmentSymbols in SentryXcodeProjectTests covers development and release builds with the option on and off. The development build with the option off fails without the fix.

🤖 Generated with Claude Code

@bitsandfoxes
bitsandfoxes force-pushed the fix/ios-upload-development-symbols branch from 4822ae4 to 7a39716 Compare November 4, 2026 15:47
@bitsandfoxes
bitsandfoxes changed the base branch from main to test/skip-android-2021-throw-frame November 4, 2026 15:59
@bitsandfoxes
bitsandfoxes marked this pull request as ready for review November 4, 2026 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant