-
-
Notifications
You must be signed in to change notification settings - Fork 337
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
[iOS, Xcode] Upload Debug Files fails on HTTP error 4xx #3551
Comments
Hi @thefranfran, Could you run the build again with this environmental variable and share the new logs? export SENTRY_CLI_EXTRA_ARGS="--force-foreground --log-level=debug" |
Hi @krystofwoldrich, Thank you for your feedback `[ -z "$WITH_ENVIRONMENT" ] && WITH_ENVIRONMENT="../node_modules/react-native/scripts/xcode/with-environment.sh" if [ -f "$WITH_ENVIRONMENT" ]; then LOCAL_NODE_BINARY=${NODE_BINARY:-node} [ -z "$SENTRY_PROPERTIES" ] && export SENTRY_PROPERTIES=sentry.properties [ -z "$SENTRY_CLI_EXECUTABLE" ] && SENTRY_CLI_PACKAGE_PATH=$("$LOCAL_NODE_BINARY" --print "require('path').dirname(require.resolve('@sentry/cli/package.json'))") [[ $SENTRY_INCLUDE_NATIVE_SOURCES == "true" ]] && INCLUDE_SOURCES_FLAG="--include-sources" || INCLUDE_SOURCES_FLAG="" EXTRA_ARGS="--force-foreground --log-level=debug $SENTRY_CLI_DEBUG_FILES_UPLOAD_EXTRA_ARGS $INCLUDE_SOURCES_FLAG" UPLOAD_DEBUG_FILES=""$SENTRY_CLI_EXECUTABLE" debug-files upload $EXTRA_ARGS "$DWARF_DSYM_FOLDER_PATH"" if [ "$SENTRY_DISABLE_AUTO_UPLOAD" != true ]; then I do not know if it will help you so tell me if it doesn't |
@thefranfran It seems like the output didn't change, how does your |
my export NODE_BINARY=$(command -v node) and my export NODE_BINARY=/opt/homebrew/bin/node |
Hi, I have similar issue. After upgrading to Expo 50 and migrating from sentry-expo to @sentry/react-native the build (using remote EAS Build) started failing with the next error:
These are the XCode logs:
|
Also seeing the exact same issue as @alterx, breaking on eas staging and production builds, can this issue be reopened until there is a resolution? |
Happening again today. |
still happens on the last version! |
Same here. |
Is anyone experiencing intermittent errors with HTTP status 403? We would like to solve any problem folks are having. To do that, please help us by providing some further context like: Your org slug. Times when it happened. How are you passing the org slug, auth token etc? If your issue is not related to 403's, please raise a specific issue describing what the error is. Including debug logs ( |
yes
…On Tue, Sep 10, 2024 at 11:34 AM Bruno Garcia ***@***.***> wrote:
Is anyone experiencing intermittent errors with HTTP status 403?
If so, please comment/engage with:
- getsentry/sentry-javascript-bundler-plugins#605 <getsentry/sentry-javascript-bundler-plugins#605>
We would like to solve any problem folks are having. To do that, please
help us by providing some further context like: Your org slug. Times when
it happened. How are you passing the org slug, auth token etc?
If your issue is not related to 403's, please raise a specific issue
describing what the error is. Including debug logs and the other info I
mentioned is hugely helpful for us to track down what's going on.
—
Reply to this email directly, view it on GitHub
<#3551 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAEVCOI5OWYW5I2YEAHJZTZV43UHAVCNFSM6AAAAABCKKQUPKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBRG4ZTMNJSHA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
I you are receiving 403 during the Debug Files Upload it's possible that the IP from where you are uploading the files is in range blocked by Google Cloud and so the request doesn't reach our servers. To disable source maps auto upload, set SENTRY_DISABLE_AUTO_UPLOAD=true in your environment variables. Or to allow failing upload, set SENTRY_ALLOW_FAILURE=true. If the upload fails you will have to upload the files manually following our guides -> https://docs.sentry.io/platforms/react-native/sourcemaps/ |
I think it is quite unacceptable that a service makes uploading the source code and symbols (a non critical task) fail the entire build if the step fails. The same happens if some error like It's effectively making thousands of build processes dependent on Sentry's service availability. This should be a warning, not a failure, and the default should be the other way arround (eg Furthermore, I am working with multiple organizations, which have different access tokens, making the secret management quite challenging for testing local builds (ie, need to constantly change the token depending on the project, othewise step will fail) |
100%
…On Wed, Sep 18, 2024 at 3:52 PM Sérgio ***@***.***> wrote:
I think it is quite unacceptable that a service makes uploading the source
code and symbols (a non critical task) fail the entire build if the step
fails. The same happens if some error like Project not found. Please check
that you entered the project and organization slugs correctly.
It's effectively making thousands of build processes dependent on Sentry's
service availability.
This should be a warning, not a failure, and the default should be the
other way arround (eg SENTRY_ABORT_BUILD_IF_FAIL or something similar)
Furthermore, I am working with multiple organizations, which have
different access tokens, making the secret management quite challenging.
—
Reply to this email directly, view it on GitHub
<#3551 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAEVCJEA7XNSURIQB5HEN3ZXH72RAVCNFSM6AAAAABCKKQUPKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJZGU4DQNZUGM>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Returned to needs discussion to look at #3551 (comment) |
Thank you @sergioisidoro, We understand that by failing the build process we are creating a dependency on Sentry. The reason is delivering the best error experience we can. That means ensuring source maps are uploaded and thus errors are symbolicated and correctly grouped into issues. If source maps upload would be only a warning, this could cause situations where if upload would fail and the app is unknowingly release without the source map uploaded new error would be created which would not be symbolicated, this could have larger consequences like triggering alerts and more. We always include the message We close this issue as the original source of the 4xx error was resolved. |
OS:
Platform:
SDK:
@sentry/react-native
(>= 1.0.0)react-native-sentry
(<= 0.43.2)SDK version: 5.17.0
react-native
version: 0.73.2Are you using Expo?
Are you using sentry.io or on-premise?
Configuration:
(
@sentry/react-native
)I have the following issue:
I installed the package with command
npx @sentry/wizard@latest -s -i reactNative
and the metro config is setup aswell
It's look like it's an error during the execution of the Run Script Phase which is
Upload Debug Symbols to Sentry
And the script is
/bin/sh ../node_modules/@sentry/react-native/scripts/sentry-xcode-debug-files.sh
Steps to reproduce:
npx @sentry/wizard@latest -s -i reactNative
Actual result:
Command PhaseScriptExecution failed with a nonzero exit code
PhaseScriptExecution Upload\ Debug\ Symbols\ to\ Sentry
BuildFail.txt
The text was updated successfully, but these errors were encountered: