Skip to content

Commit 9cd267b

Browse files
committed
Add userId and deviceId to rageshake data.
1 parent 0fbf799 commit 9cd267b

File tree

2 files changed

+6
-11
lines changed

2 files changed

+6
-11
lines changed

features/rageshake/impl/build.gradle.kts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ dependencies {
3838
implementation(projects.libraries.architecture)
3939
implementation(projects.libraries.designsystem)
4040
implementation(projects.libraries.uiStrings)
41+
implementation(projects.libraries.sessionStorage.api)
4142
api(libs.squareup.seismic)
4243
api(projects.features.rageshake.api)
4344
implementation(libs.androidx.datastore.preferences)

features/rageshake/impl/src/main/kotlin/io/element/android/features/rageshake/impl/reporter/DefaultBugReporter.kt

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ import io.element.android.libraries.core.mimetype.MimeTypes
3636
import io.element.android.libraries.di.AppScope
3737
import io.element.android.libraries.di.ApplicationContext
3838
import io.element.android.libraries.network.useragent.UserAgentProvider
39+
import io.element.android.libraries.sessionstorage.api.SessionStore
3940
import kotlinx.coroutines.flow.first
4041
import kotlinx.coroutines.withContext
4142
import okhttp3.Call
@@ -66,8 +67,8 @@ class DefaultBugReporter @Inject constructor(
6667
private val coroutineDispatchers: CoroutineDispatchers,
6768
private val okHttpClient: Provider<OkHttpClient>,
6869
private val userAgentProvider: UserAgentProvider,
70+
private val sessionStore: SessionStore,
6971
/*
70-
private val activeSessionHolder: ActiveSessionHolder,
7172
private val versionProvider: VersionProvider,
7273
private val vectorPreferences: VectorPreferences,
7374
private val vectorFileLogger: VectorFileLogger,
@@ -198,18 +199,11 @@ class DefaultBugReporter @Inject constructor(
198199
?.let { gzippedFiles.add(it) }
199200
*/
200201

201-
var deviceId = "undefined"
202-
var userId = "undefined"
202+
val sessionData = sessionStore.getLatestSession()
203+
val deviceId = sessionData?.deviceId ?: "undefined"
204+
val userId = sessionData?.userId ?: "undefined"
203205
var olmVersion = "undefined"
204206

205-
/*
206-
activeSessionHolder.getSafeActiveSession()?.let { session ->
207-
userId = session.myUserId
208-
deviceId = session.sessionParams.deviceId ?: "undefined"
209-
olmVersion = session.cryptoService().getCryptoVersion(context, true)
210-
}
211-
*/
212-
213207
if (!mIsCancelled) {
214208
val text = when (reportType) {
215209
ReportType.BUG_REPORT -> bugDescription

0 commit comments

Comments
 (0)