Skip to content
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.

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

Rework upload symbol script #4328

Closed
armcknight opened this issue Sep 12, 2024 · 3 comments · Fixed by #4398
Closed

Rework upload symbol script #4328

armcknight opened this issue Sep 12, 2024 · 3 comments · Fixed by #4398

Comments

@armcknight
Copy link
Member

armcknight commented Sep 12, 2024

Description

We currently require an auth token to be provided, and caution against committing the patch to apply to get the symbol upload script to avoid committing auth tokens to the repo:

  • ## Upload iOS-Swift's dSYMs with Xcode Run Script
    The following script applies a patch so Xcode uploads the iOS-Swift's dSYMs to Sentry during Xcode's build phase.
    Ensure to not commit the patch file after running this script, which then contains your auth token.
    ```sh
    ./scripts/upload-dsyms-with-xcode-build-phase.sh YOUR_AUTH_TOKEN
    ```
  • # Use this script to apply a patch so Xcode uploads the iOS-Swift's dSYMs to
    # Sentry during Xcode's build phase.
    # Ensure to not commit the patch file after running this script, which then contains
    # your auth token.

This can now be done using environment variables, so we should rework the script and leave it in the build phases. See the script I wrote for the SE iOS demo: https://github.com/sentry-demos/ios/blob/master/upload-symbols.sh

Ideally, we create a new build config for debug builds that upload symbols. This way, we don't upload a large volume from the standard debug config. We can also configure it to always upload for release builds.

This way developers can freely upload symbols as needed with minimal changes to the git working index.

@philipphofmann
Copy link
Member

@armcknight, do you use the upload-dsyms-with-xcode-build-phase.sh? I created this some time ago because I regularly upload the dSYMs via Xcode. Nowadays, I always check symbolication via TestFlight and have even forgotten that this script exists. @brustolin also never uses it. My point is if none of us use it, we don't need to improve it, and we can even discuss removing it.

@armcknight
Copy link
Member Author

Yep, that's fair too, we could consider removing it. It's been a long time since I needed something symbolicated for testing while debugging.

philipphofmann added a commit that referenced this issue Oct 3, 2024
We remove this script because there is a risk of accidentially
committing tokens to GH, and we're not using this script anymore.

Fixes GH-4328
@philipphofmann
Copy link
Member

Excellent. I removed it: #4398.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants