From 580312e4c79db6bacc7f31c35d8abbfd2ee9639b Mon Sep 17 00:00:00 2001
From: Harry <109690906+harry-signal@users.noreply.github.com>
Date: Mon, 27 Mar 2023 17:09:39 -0700
Subject: [PATCH] Remove todo for update account attributes error

---
 .../Registration/RegistrationCoordinatorImpl+Service.swift   | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/Signal/Registration/RegistrationCoordinatorImpl+Service.swift b/Signal/Registration/RegistrationCoordinatorImpl+Service.swift
index 15e50c7779..f6f2b40bf5 100644
--- a/Signal/Registration/RegistrationCoordinatorImpl+Service.swift
+++ b/Signal/Registration/RegistrationCoordinatorImpl+Service.swift
@@ -307,8 +307,9 @@ extension RegistrationCoordinatorImpl {
             return signalService.urlSessionForMainSignalService().promiseForTSRequest(request)
                 .map(on: schedulers.sync) { response in
                     guard response.responseStatusCode >= 200, response.responseStatusCode < 300 else {
-                        // TODO[Registration]: what other error codes can come up here?
-                        return OWSAssertionError("Got unexpected response code from update attributes request.")
+                        // Errors are undifferentiated; the only actual error we can get is an unauthenticated
+                        // one and there isn't any way to handle that as different from a, say server 500.
+                        return OWSAssertionError("Got unexpected response code from update attributes request: \(response.responseStatusCode).")
                     }
                     return nil
                 }