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

Envelope with offline storage can't reupload #2373

Closed
wangbax opened this issue Nov 17, 2022 · 2 comments · Fixed by #2377
Closed

Envelope with offline storage can't reupload #2373

wangbax opened this issue Nov 17, 2022 · 2 comments · Fixed by #2377
Assignees
Labels
Platform: Android Type: Bug Something isn't working

Comments

@wangbax
Copy link

wangbax commented Nov 17, 2022

Integration

sentry-android

Build System

Gradle

AGP Version

7.0.2

Proguard

Disabled

Version

6.7.0

Steps to Reproduce

  1. close network
  2. open app
  3. close app
  4. reopen app

Expected Result

Adding all envelope to offline storage: /data/user/0/[package]/cache/sentry/[dsn]/[id].envelope

Actual Result

Adding all envelope to offline storage: /data/user/0/[package]/cache/sentry/[id].envelope

image

@wangbax
Copy link
Author

wangbax commented Nov 17, 2022

This issues should be related to new AndroidEnvelopeCache(options) #2277

imageimage

My solution:

SentryAndroid.init(context) { options ->
      options.apply {
        this.dsn = dsn
        // after change dsn, try reset disk cache
        setEnvelopeDiskCache(AndroidEnvelopeCache(options))
      }
    }

Hope can fix this issue least version. ☺️

@markushi markushi self-assigned this Nov 17, 2022
@romtsn romtsn added Type: Bug Something isn't working and removed Status: Untriaged labels Nov 17, 2022
@romtsn romtsn moved this to Needs Discussion in Mobile & Cross Platform SDK Nov 17, 2022
@romtsn romtsn moved this from Needs Discussion to In Progress in Mobile & Cross Platform SDK Nov 17, 2022
Repository owner moved this from In Progress to Done in Mobile & Cross Platform SDK Nov 18, 2022
@markushi
Copy link
Member

FYI: The fix will be part of the 6.8.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Platform: Android Type: Bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants