Skip to content

Conversation

@lucas-zimerman
Copy link
Collaborator

@lucas-zimerman lucas-zimerman commented Jan 21, 2026

Due to not replacing the defaultIntegration, we have a side effect that the sibling SDK will add their own integrations, inserting the BrowserSession automatically. This is wrong since both native and the JS implementation has auto session implementation.

The correct fix was applied by adding the capacitor integrations into defaultIntegrations, overwriting the sibling SDK integrations and thus, only having a single session being created.

A minor refactor was also implemented where I renamed finalOption to sharedOption since it matches what it is actually being used, as a shared data instead of the final option being used.

Before:
image

After:
image

image

@github-actions
Copy link
Contributor

github-actions bot commented Jan 21, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Bug Fixes 🐛

  • (V3) Duplicated session by lucas-zimerman in #1088

Build / dependencies / internal 🔧

  • (V3) Bump sentry cocoa v9 by lucas-zimerman in #1086
  • Use pull_request_target for changelog preview by BYK in #1079

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 21, 2026

Fails
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

### Fixes

- Duplicated session ([#1088](https://github.com/getsentry/sentry-capacitor/pull/1088))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against fc2a7a5

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants