mirrors/bitwarden-android
1
0
Fork 0
mirror of https://github.com/bitwarden/android.git synced 2024-10-31 07:05:35 +03:00
Code Issues Projects Releases Packages Wiki Activity

PM-13726: Process cipher notifications without organizationIds or collectionIds (#4102)

Browse source
This commit is contained in:
David Perez 2024-10-16 11:43:46 -05:00 committed by GitHub
parent 62cfd5e746
commit cf3624264e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 4 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
app/src/main/java/com/x8bit/bitwarden/data/platform/manager/PushManagerImpl.kt
View file

@ -162,19 +162,14 @@ class PushManagerImpl @Inject constructor(
string = notification.payload,
)
.takeIf { isLoggedIn(userId) && it.userMatchesNotification(userId) }
?.takeIf {
it.cipherId != null &&
it.revisionDate != null &&
it.organizationId != null &&
it.collectionIds != null
}
?.takeIf { it.cipherId != null && it.revisionDate != null }
?.let {
mutableSyncCipherUpsertSharedFlow.tryEmit(
SyncCipherUpsertData(
cipherId = requireNotNull(it.cipherId),
revisionDate = requireNotNull(it.revisionDate),
organizationId = requireNotNull(it.organizationId),
collectionIds = requireNotNull(it.collectionIds),
organizationId = it.organizationId,
collectionIds = it.collectionIds,
isUpdate = type == NotificationType.SYNC_CIPHER_UPDATE,
),
)
@ -339,6 +334,6 @@ class PushManagerImpl @Inject constructor(
): Boolean = authDiskSource.getAccountTokens(userId)?.isLoggedIn == true
}
private fun NotificationPayload.userMatchesNotification(userId: String?): Boolean {
private fun NotificationPayload.userMatchesNotification(userId: String): Boolean {
return this.userId != null && this.userId == userId
}