From 591ec1052c43dcbf89df223911dedae4e4012b8b Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Mon, 18 Nov 2024 13:04:39 +0100 Subject: [PATCH] Ensure m.call.notify event is not filtered out by push rules. --- .../sdk/api/session/pushrules/RuleIds.kt | 3 + .../pushrules/DefaultPushRuleService.kt | 2 +- .../pushrules/LocalElementCallPushRule.kt | 68 +++++++++++++++++++ 3 files changed, 72 insertions(+), 1 deletion(-) create mode 100644 matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/pushrules/LocalElementCallPushRule.kt diff --git a/matrix-sdk-android/src/main/java/org/matrix/android/sdk/api/session/pushrules/RuleIds.kt b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/api/session/pushrules/RuleIds.kt index 34581b613a..c244d02ff2 100644 --- a/matrix-sdk-android/src/main/java/org/matrix/android/sdk/api/session/pushrules/RuleIds.kt +++ b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/api/session/pushrules/RuleIds.kt @@ -57,6 +57,9 @@ object RuleIds { const val RULE_ID_POLL_END = ".m.rule.poll_end" const val RULE_ID_POLL_END_UNSTABLE = ".org.matrix.msc3930.rule.poll_end" + const val RULE_ID_ELEMENT_CALL_NOTIFY = ".m.call.notify" + const val RULE_ID_ELEMENT_CALL_NOTIFY_UNSTABLE = ".org.matrix.msc4075.call.notify" + // Not documented const val RULE_ID_FALLBACK = ".m.rule.fallback" diff --git a/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/pushrules/DefaultPushRuleService.kt b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/pushrules/DefaultPushRuleService.kt index cdbb8f16a3..73507c120f 100644 --- a/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/pushrules/DefaultPushRuleService.kt +++ b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/pushrules/DefaultPushRuleService.kt @@ -103,7 +103,7 @@ internal class DefaultPushRuleService @Inject constructor( } return RuleSet( - content = contentRules, + content = contentRules.withElementCallPushRules(), override = overrideRules, room = roomRules, sender = senderRules, diff --git a/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/pushrules/LocalElementCallPushRule.kt b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/pushrules/LocalElementCallPushRule.kt new file mode 100644 index 0000000000..2606248042 --- /dev/null +++ b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/pushrules/LocalElementCallPushRule.kt @@ -0,0 +1,68 @@ +/* + * Copyright (c) 2024 The Matrix.org Foundation C.I.C. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.matrix.android.sdk.internal.session.pushrules + +import org.matrix.android.sdk.api.session.pushrules.Action +import org.matrix.android.sdk.api.session.pushrules.RuleIds +import org.matrix.android.sdk.api.session.pushrules.rest.PushCondition +import org.matrix.android.sdk.api.session.pushrules.rest.PushRule + +private val localElementCallPushRule = PushRule( + ruleId = RuleIds.RULE_ID_ELEMENT_CALL_NOTIFY, + conditions = listOf( + PushCondition( + kind = "event_match", + key = "type", + pattern = "m.call.notify", + ) + ), + actions = listOf( + Action.ACTION_NOTIFY, + ), + enabled = true, +) + +private val localElementCallPushRuleUnstable = PushRule( + ruleId = RuleIds.RULE_ID_ELEMENT_CALL_NOTIFY_UNSTABLE, + conditions = listOf( + PushCondition( + kind = "event_match", + key = "type", + pattern = "org.matrix.msc4075.call.notify", + ) + ), + actions = listOf( + Action.ACTION_NOTIFY, + ), + enabled = true, +) + +/** + * Ensure that the element call push rules are present. + */ +fun List.withElementCallPushRules(): List { + val ruleIds = map { it.ruleId } + return buildList { + addAll(this@withElementCallPushRules) + if (!ruleIds.contains(localElementCallPushRule.ruleId)) { + add(localElementCallPushRule) + } + if (!ruleIds.contains(localElementCallPushRuleUnstable.ruleId)) { + add(localElementCallPushRuleUnstable) + } + } +}