-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
feat: Update JavaScript SDKs to v8.48.0 #1046
Conversation
@timfish sounds like this is the PR that will allow ANR events to symbolicate? Would a client on SDK v5.9.0 need to make any changes on their side? Or will it work once this is merged? Thanks for all you've done! |
This needs to be merged and then this will be released in v5.10.0 |
@@ -55,7 +55,7 @@ jobs: | |||
# we want that the matrix keeps running, default is to cancel all if one fails. | |||
fail-fast: false | |||
matrix: | |||
os: [ubuntu-latest, macos-latest, windows-latest] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why has this changed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ubuntu-latest
changed to Ubuntu 24 in the last week and all the Linux tests started failing due to sandbox issues:
https://github.com/getsentry/sentry-electron/actions/runs/12652009781
The SUID sandbox helper binary was found, but is not configured correctly. Rather than run without sandboxing I'm aborting now. You need to make sure that /home/runner/work/sentry-electron/sentry-electron/.cache/30.5.1-x64/chrome-sandbox is owned by root and has mode 4755.
We already had to disable the sandbox for older versions of Electron but something has changed in the latest Github Actions images
sentry-electron/test/e2e/context.ts
Lines 75 to 77 in 976b580
if (process.platform === 'linux' && (version.major || 0) < 13) { | |
args.push('--no-sandbox'); | |
} |
We cannot disable the sandbox entirely on Linux because we need to test it's functionality!
I need to find a way to fix this but it's not important for this PR
I updated branch protection to require ubuntu-22.04 instead of ubuntu-latest! |
Updates JavaScript SDKs to v8.48.0.
The Node SDK has a fix for ANR debug IDs so this PR also includes test changes to confirm this.