add FoxchatError type

Foxchat.Core/Database/IDatabaseContext.cs

@@ -29,7 +29,7 @@ public abstract class IDatabaseContext : DbContext
     public async Task<Instance> GetInstanceAsync(CancellationToken ct = default)
     {
         var instance = await Instance.FirstOrDefaultAsync(ct)
-            ?? throw new Exception("GetInstanceAsync called without Instance being initialized"); // TODO: replace this with specific exception type
+            ?? throw new FoxchatError.DatabaseError("GetInstanceAsync called without Instance being initialized");
         return instance;
     }
 

Foxchat.Core/Federation/RequestSigningService.Client.cs

@@ -1,4 +1,5 @@
 using System.Net.Http.Headers;
+using Foxchat.Core.Models;
 using Newtonsoft.Json;
 using Newtonsoft.Json.Serialization;
 
@@ -9,9 +10,6 @@ public partial class RequestSigningService
     public const string USER_AGENT_HEADER = "User-Agent";
     public const string USER_AGENT = "Foxchat.NET";
     public const string DATE_HEADER = "Date";
-    public const string CONTENT_LENGTH_HEADER = "Content-Length";
-    public const string CONTENT_TYPE_HEADER = "Content-Type";
-    public const string CONTENT_TYPE = "application/json; charset=utf-8";
 
     public const string SERVER_HEADER = "X-Foxchat-Server";
     public const string SIGNATURE_HEADER = "X-Foxchat-Signature";
@@ -28,19 +26,16 @@ public partial class RequestSigningService
     public async Task<T> RequestAsync<T>(HttpMethod method, string domain, string requestPath, string? userId = null, object? body = null)
     {
         var request = BuildHttpRequest(method, domain, requestPath, userId, body);
-        _logger.Debug("Content length in header: '{ContentLength}'", request.Headers.Where(c => c.Key == "Content-Length"));
         var resp = await _httpClient.SendAsync(request);
         if (!resp.IsSuccessStatusCode)
         {
             var error = await resp.Content.ReadAsStringAsync();
-            _logger.Error("Received {Status}, body: {Error}", resp.StatusCode, error);
-            // TODO: replace this with specific exception type
-            throw new Exception("oh no a request error");
+            throw new FoxchatError.OutgoingFederationError($"Request to {domain}/{requestPath} returned an error", DeserializeObject<ApiError>(error));
         }
 
         var bodyString = await resp.Content.ReadAsStringAsync();
-        // TODO: replace this with specific exception type
-        return DeserializeObject<T>(bodyString) ?? throw new Exception("oh no invalid json");
+        return DeserializeObject<T>(bodyString)
+            ?? throw new FoxchatError.OutgoingFederationError($"Request to {domain}/{requestPath} returned invalid response body");
     }
 
     private HttpRequestMessage BuildHttpRequest(HttpMethod method, string domain, string requestPath, string? userId = null, object? bodyData = null)

Foxchat.Core/Federation/RequestSigningService.cs

@@ -27,7 +27,7 @@ public partial class RequestSigningService(ILogger logger, IClock clock, IDataba
         formatter.SetHashAlgorithm(nameof(SHA256));
         var signature = formatter.CreateSignature(hash);
 
-        _logger.Debug("Generated signature for {Host} {RequestPath}: {Signature}", data.Host, data.RequestPath, WebEncoders.Base64UrlEncode(signature));
+        _logger.Debug("Generated signature for {Host} {RequestPath}", data.Host, data.RequestPath);
         return WebEncoders.Base64UrlEncode(signature);
     }
 
@@ -41,13 +41,11 @@ public partial class RequestSigningService(ILogger logger, IClock clock, IDataba
         var time = ParseTime(dateHeader);
         if ((now + Duration.FromMinutes(1)) < time)
         {
-            // TODO: replace this with specific exception type
-            throw new Exception("Request was made in the future");
+            throw new FoxchatError.IncomingFederationError("Request was made in the future");
         }
         else if ((now - Duration.FromMinutes(1)) > time)
         {
-            // TODO: replace this with specific exception type
-            throw new Exception("Request was made too long ago");
+            throw new FoxchatError.IncomingFederationError("Request was made too long ago");
         }
 
         var plaintext = GeneratePlaintext(new SignatureData(time, host, requestPath, contentLength, userId));
@@ -67,8 +65,6 @@ public partial class RequestSigningService(ILogger logger, IClock clock, IDataba
         var contentLength = data.ContentLength != null ? data.ContentLength.ToString() : "";
         var userId = data.UserId ?? "";
 
-        Log.Information("Plaintext string: {Plaintext}", $"{time}:{data.Host}:{data.RequestPath}:{contentLength}:{userId}");
-
         return $"{time}:{data.Host}:{data.RequestPath}:{contentLength}:{userId}";
     }
 

Foxchat.Core/Federation/SignatureData.cs

@@ -8,4 +8,4 @@ public record SignatureData(
     string RequestPath,
     int? ContentLength,
     string? UserId
-) { }
+);

Foxchat.Core/FoxchatError.cs

@@ -0,0 +1,15 @@
+namespace Foxchat.Core;
+
+public class FoxchatError(string message) : Exception(message)
+{
+    public class ApiError(string message) : FoxchatError(message);
+    public class BadRequest(string message) : ApiError(message);
+    public class IncomingFederationError(string message) : FoxchatError(message);
+
+    public class OutgoingFederationError(string message, Models.ApiError? innerError = null) : FoxchatError(message)
+    {
+        public Models.ApiError? InnerError => innerError;
+    }
+
+    public class DatabaseError(string message) : FoxchatError(message);
+}

Foxchat.Core/Models/ApiError.cs

@@ -0,0 +1,17 @@
+namespace Foxchat.Core.Models;
+
+public record ApiError(int Status, ErrorCode Code, string Message);
+
+public enum ErrorCode
+{
+    INTERNAL_SERVER_ERROR,
+    OBJECT_NOT_FOUND,
+    INVALID_SERVER,
+    INVALID_HEADER,
+    INVALID_DATE,
+    INVALID_SIGNATURE,
+    MISSING_SIGNATURE,
+    GUILD_NOT_FOUND,
+    UNAUTHORIZED,
+    INVALID_REST_EVENT,
+}

Foxchat.Core/UlidConverter.cs

@@ -6,5 +6,4 @@ namespace Foxchat.Core;
 public class UlidConverter() : ValueConverter<Ulid, Guid>(
     convertToProviderExpression: x => x.ToGuid(),
     convertFromProviderExpression: x => new Ulid(x)
-)
-{ }
+);