From 9afc94c050788930948d3fdb92ae1ec6a1eebc54 Mon Sep 17 00:00:00 2001 From: Vitor Pamplona Date: Sun, 2 Apr 2023 11:32:38 -0400 Subject: [PATCH] Display relay info in spam message logs --- .../java/com/vitorpamplona/amethyst/model/AntiSpamFilter.kt | 5 +++-- .../java/com/vitorpamplona/amethyst/model/LocalCache.kt | 6 +++--- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/app/src/main/java/com/vitorpamplona/amethyst/model/AntiSpamFilter.kt b/app/src/main/java/com/vitorpamplona/amethyst/model/AntiSpamFilter.kt index 001069c08..34b26bba3 100644 --- a/app/src/main/java/com/vitorpamplona/amethyst/model/AntiSpamFilter.kt +++ b/app/src/main/java/com/vitorpamplona/amethyst/model/AntiSpamFilter.kt @@ -4,6 +4,7 @@ import android.util.Log import android.util.LruCache import androidx.lifecycle.LiveData import com.vitorpamplona.amethyst.service.model.Event +import com.vitorpamplona.amethyst.service.relays.Relay import com.vitorpamplona.amethyst.ui.components.BundledUpdate import kotlinx.coroutines.Dispatchers @@ -14,7 +15,7 @@ class AntiSpamFilter { val spamMessages = LruCache(1000) @Synchronized - fun isSpam(event: Event): Boolean { + fun isSpam(event: Event, relay: Relay?): Boolean { val idHex = event.id // if short message, ok @@ -27,7 +28,7 @@ class AntiSpamFilter { val hash = (event.content + event.tags.flatten().joinToString(",")).hashCode() if ((recentMessages[hash] != null && recentMessages[hash] != idHex) || spamMessages[hash] != null) { - Log.w("Potential SPAM Message", "${event.id} ${recentMessages[hash]} ${spamMessages[hash] != null} ${event.content.replace("\n", " | ")}") + Log.w("Potential SPAM Message", "${event.id} ${recentMessages[hash]} ${spamMessages[hash] != null} ${relay?.url} ${event.content.replace("\n", " | ")}") // Log down offenders if (spamMessages.get(hash) == null) { diff --git a/app/src/main/java/com/vitorpamplona/amethyst/model/LocalCache.kt b/app/src/main/java/com/vitorpamplona/amethyst/model/LocalCache.kt index da8686412..eed0b602b 100644 --- a/app/src/main/java/com/vitorpamplona/amethyst/model/LocalCache.kt +++ b/app/src/main/java/com/vitorpamplona/amethyst/model/LocalCache.kt @@ -187,7 +187,7 @@ object LocalCache { // Already processed this event. if (note.event != null) return - if (antiSpam.isSpam(event)) { + if (antiSpam.isSpam(event, relay)) { relay?.let { it.spamCounter++ } @@ -223,7 +223,7 @@ object LocalCache { // Already processed this event. if (note.event?.id() == event.id()) return - if (antiSpam.isSpam(event)) { + if (antiSpam.isSpam(event, relay)) { relay?.let { it.spamCounter++ } @@ -543,7 +543,7 @@ object LocalCache { // Already processed this event. if (note.event != null) return - if (antiSpam.isSpam(event)) { + if (antiSpam.isSpam(event, relay)) { relay?.let { it.spamCounter++ }