Skip to content

Commit 08c3ecc

Browse files
committed
Fix: bug that causes onesignalId to be local when setting alias while identity verification is off
1 parent f4a9356 commit 08c3ecc

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/internal/OneSignalImp.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -471,7 +471,7 @@ internal class OneSignalImp : IOneSignal, IServiceProvider {
471471
modify(identityModel, propertiesModel)
472472
}
473473

474-
if (identityModel.jwtToken != null) {
474+
if (!useIdentityVerification || identityModel.jwtToken != null) {
475475
setupNewSubscription(identityModel, propertiesModel, suppressBackendOperation, sdkId)
476476
}
477477

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/user/internal/operations/impl/executors/IdentityOperationExecutor.kt

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package com.onesignal.user.internal.operations.impl.executors
33
import com.onesignal.common.NetworkUtils
44
import com.onesignal.common.exceptions.BackendException
55
import com.onesignal.common.modeling.ModelChangeTags
6+
import com.onesignal.core.internal.config.ConfigModelStore
67
import com.onesignal.core.internal.operations.ExecutionResponse
78
import com.onesignal.core.internal.operations.ExecutionResult
89
import com.onesignal.core.internal.operations.IOperationExecutor
@@ -19,6 +20,7 @@ import com.onesignal.user.internal.operations.impl.states.NewRecordsState
1920
internal class IdentityOperationExecutor(
2021
private val _identityBackend: IIdentityBackendService,
2122
private val _identityModelStore: IdentityModelStore,
23+
private val _configModelStore: ConfigModelStore,
2224
private val _buildUserService: IRebuildUserService,
2325
private val _newRecordState: NewRecordsState,
2426
) : IOperationExecutor {
@@ -45,7 +47,13 @@ internal class IdentityOperationExecutor(
4547

4648
if (lastOperation is SetAliasOperation) {
4749
try {
48-
val identityAlias = _identityModelStore.getIdentityAlias()
50+
var identityAlias: Pair<String, String>
51+
// use onesignalId from the operation if identity verification is turned off
52+
if (_configModelStore.model.useIdentityVerification) {
53+
identityAlias = _identityModelStore.getIdentityAlias()
54+
} else {
55+
identityAlias = Pair(IdentityConstants.ONESIGNAL_ID, lastOperation.onesignalId)
56+
}
4957
_identityBackend.setAlias(
5058
lastOperation.appId,
5159
identityAlias.first,

0 commit comments

Comments
 (0)