Skip to content

fix(mobile-app): Avoid panic if chunk uploading not supported#2567

Merged
szokeasaurusrex merged 1 commit intomasterfrom
szokeasaurusrex/panic-chunk-upload
Jun 25, 2025
Merged

fix(mobile-app): Avoid panic if chunk uploading not supported#2567
szokeasaurusrex merged 1 commit intomasterfrom
szokeasaurusrex/panic-chunk-upload

Conversation

@szokeasaurusrex
Copy link
Copy Markdown
Member

Comment on lines +276 to +277
const SELF_HOSTED_ERROR_HINT: &str = "If you are using a self-hosted Sentry server, \
update to the latest version of Sentry to use the mobile-app upload command.";
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see the point of extracting this specific part of the error message

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, just saw the other PR.

@szokeasaurusrex szokeasaurusrex merged commit 08c81aa into master Jun 25, 2025
18 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/panic-chunk-upload branch June 25, 2025 15:06
szokeasaurusrex added a commit that referenced this pull request Oct 3, 2025
szokeasaurusrex added a commit that referenced this pull request Oct 3, 2025
### Description
Add a weekly action, which will create issues if any security
vulnerabilities are detected in our dependencies.

Action was adopted from
[`sentry-rust`](https://github.com/getsentry/sentry-rust/blob/9ba9a6452df7b8e828030f4d91ebd0c6664b6707/.github/workflows/weekly.yml#L33-L43)

### Issues
Fixes #2567
Fixes CLI-119

<!--
#### Reminders
- Add GH Issue ID _&_ Linear ID (if applicable)
- PR title should use [conventional
commit](https://develop.sentry.dev/engineering-practices/commit-messages/#type)
style (`feat:`, `fix:`, `ref:`, `meta:`)
- For external contributors:
[CONTRIBUTING.md](https://github.com/getsentry/sentry-cli/blob/master/CONTRIBUTING.md),
[Sentry SDK development docs](https://develop.sentry.dev/sdk/), [Discord
community](https://discord.gg/Ww9hbqr)
-->
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.

2 participants