From 158f9b83fb8f6a63688d1763b0b402822c3924a9 Mon Sep 17 00:00:00 2001
From: Nik Clayton <nik@ngo.org.uk>
Date: Tue, 16 May 2023 18:24:38 +0200
Subject: [PATCH] updateMarkersWithAuth(): Require DOMAIN header (#3660)

Otherwise markers are updated for the wrong account.

Fixes https://github.com/tuskyapp/Tusky/issues/3658
---
 .../components/notifications/NotificationFetcher.kt   | 11 ++++++++---
 .../com/keylesspalace/tusky/network/MastodonApi.kt    |  1 +
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/app/src/main/java/com/keylesspalace/tusky/components/notifications/NotificationFetcher.kt b/app/src/main/java/com/keylesspalace/tusky/components/notifications/NotificationFetcher.kt
index 420dd8213..ff05fa4b6 100644
--- a/app/src/main/java/com/keylesspalace/tusky/components/notifications/NotificationFetcher.kt
+++ b/app/src/main/java/com/keylesspalace/tusky/components/notifications/NotificationFetcher.kt
@@ -115,6 +115,7 @@ class NotificationFetcher @Inject constructor(
     private fun fetchNewNotifications(account: AccountEntity): List<Notification> {
         val authHeader = String.format("Bearer %s", account.accessToken)
 
+        Log.d(TAG, "getting notification marker for ${account.fullName}")
         val minId = when (val marker = fetchMarker(authHeader, account)) {
             null -> account.lastNotificationId.takeIf { it != "0" }
             else -> if (account.lastNotificationId.isLessThan(marker.lastReadId)) marker.lastReadId else account.lastNotificationId
@@ -132,8 +133,12 @@ class NotificationFetcher @Inject constructor(
         // in the marker.
         notifications.firstOrNull()?.let {
             val newMarkerId = notifications.first().id
-            Log.d(TAG, "updating notification marker to: $newMarkerId")
-            mastodonApi.updateMarkersWithAuth(authHeader, notificationsLastReadId = newMarkerId)
+            Log.d(TAG, "updating notification marker for ${account.fullName} to: $newMarkerId")
+            mastodonApi.updateMarkersWithAuth(
+                auth = authHeader,
+                domain = account.domain,
+                notificationsLastReadId = newMarkerId
+            )
         }
 
         return notifications
@@ -147,7 +152,7 @@ class NotificationFetcher @Inject constructor(
                 listOf("notifications")
             ).blockingGet()
             val notificationMarker = allMarkers["notifications"]
-            Log.d(TAG, "Fetched marker: $notificationMarker")
+            Log.d(TAG, "Fetched marker for ${account.fullName}: $notificationMarker")
             notificationMarker
         } catch (e: Exception) {
             Log.e(TAG, "Failed to fetch marker", e)
diff --git a/app/src/main/java/com/keylesspalace/tusky/network/MastodonApi.kt b/app/src/main/java/com/keylesspalace/tusky/network/MastodonApi.kt
index 0346ee04f..4a7162d16 100644
--- a/app/src/main/java/com/keylesspalace/tusky/network/MastodonApi.kt
+++ b/app/src/main/java/com/keylesspalace/tusky/network/MastodonApi.kt
@@ -156,6 +156,7 @@ interface MastodonApi {
     @POST("api/v1/markers")
     fun updateMarkersWithAuth(
         @Header("Authorization") auth: String,
+        @Header(DOMAIN_HEADER) domain: String,
         @Field("home[last_read_id]") homeLastReadId: String? = null,
         @Field("notifications[last_read_id]") notificationsLastReadId: String? = null
     ): NetworkResult<Unit>