Merge pull request #70 from wiverson/change-from-internal

Minor Unity Related Fixes

Author: acupofjose

Postgrest/Exceptions/FailureHint.cs

@@ -15,7 +15,7 @@ public enum Reason
 			NotAuthorized,
 			ForeignKeyViolation,
 			UniquenessViolation,
-			Internal,
+			ServerError,
 			UndefinedTable,
 			UndefinedFunction,
 			InvalidArgument
@@ -34,7 +34,7 @@ 404 when pgex.Content.Contains("42883") => UndefinedTable,
 				404 when pgex.Content.Contains("42P01") => UndefinedFunction,
 				409 when pgex.Content.Contains("23503") => ForeignKeyViolation,
 				409 when pgex.Content.Contains("23505") => UniquenessViolation,
-				500 => Internal,
+				500 => ServerError,
 				_ => Unknown
 			};
 		}

Postgrest/Helpers.cs

@@ -22,6 +22,8 @@ internal static class Helpers
 	{
 		private static readonly HttpClient Client = new HttpClient();
 
+		private static readonly Guid AppSession = Guid.NewGuid();
+
 		/// <summary>
 		/// Helper to make a request using the defined parameters to an API Endpoint and coerce into a model. 
 		/// </summary>
@@ -137,7 +139,18 @@ public static Dictionary<string, string> PrepareRequestHeaders(HttpMethod method
 
 			if (!headers.ContainsKey("X-Client-Info"))
 			{
-				headers.Add("X-Client-Info", Util.GetAssemblyVersion(typeof(Client)));
+				try
+				{
+					// Default version to match other clients
+					// https://github.com/search?q=org%3Asupabase-community+x-client-info&type=code
+					headers.Add("X-Client-Info", $"postgrest-csharp/{Util.GetAssemblyVersion(typeof(Client))}");
+				}
+				catch (Exception)
+				{
+					// Fallback for when the version can't be found
+					// e.g. running in the Unity Editor, ILL2CPP builds, etc.
+					headers.Add("X-Client-Info", $"postgrest-csharp/session-{AppSession}");
+				}
 			}
 
 			return headers;

Postgrest/Models/BaseModel.cs

@@ -43,7 +43,7 @@ public abstract class BaseModel
 		public virtual Task Delete<T>(CancellationToken cancellationToken = default) where T : BaseModel, new()
 		{
 			if (BaseUrl == null)
-				throw new PostgrestException("`BaseUrl` should be set in the model.") { Reason = FailureHint.Reason.Internal };
+				throw new PostgrestException("`BaseUrl` should be set in the model.") { Reason = FailureHint.Reason.ServerError };
 
 			var client = new Client(BaseUrl, RequestClientOptions)
 			{