Work around a JsonValue bug (FasterXML/jackson-databind#3063)

Author: markelliot

allezgo-service/src/main/java/io/allezgo/endpoints/SyncPelotonToGarmin.java

@@ -2,10 +2,8 @@
 
 import barista.Endpoints;
 import barista.HttpMethod;
-import com.fasterxml.jackson.annotation.JsonValue;
 import com.google.common.base.Strings;
 import io.allezgo.client.HttpError;
-import io.allezgo.client.Result;
 import io.allezgo.config.Configuration;
 import io.allezgo.pipes.PelotonToTcx;
 import io.allezgo.sink.garmin.GarminActivity;
@@ -51,17 +49,15 @@ public HttpMethod method() {
     public Response call(Request request) {
         List<String> failingArgs = checkArgs(request);
         if (!failingArgs.isEmpty()) {
-            return new Response(
-                    Result.error("Some required fields were missing or empty: " + failingArgs));
+            return new Response(null, "Some required fields were missing or empty: " + failingArgs);
         }
 
         PelotonClient peloton =
                 new PelotonClient(
                         new Configuration.Peloton(request.pelotonEmail, request.pelotonPassword));
 
         if (!peloton.validateLogin()) {
-            return new Response(
-                    Result.error("Unable to login to Peloton with the provided credentials"));
+            return new Response(null, "Unable to login to Peloton with the provided credentials");
         }
 
         GarminClient garmin =
@@ -72,11 +68,10 @@ public Response call(Request request) {
                                 request.garminPelotonGearName));
 
         if (!garmin.validateLogin()) {
-            return new Response(
-                    Result.error("Unable to login to Garmin with the provided credentials"));
+            return new Response(null, "Unable to login to Garmin with the provided credentials");
         }
 
-        return new Response(Result.ok(syncLast30Days(peloton, garmin)));
+        return new Response(syncLast30Days(peloton, garmin), null);
     }
 
     private static List<String> checkArgs(Request request) {
@@ -114,7 +109,7 @@ public record SyncRecord(
             String garminLink,
             boolean wasCreated) {}
 
-    public record Response(@JsonValue Result<List<SyncRecord>, String> response) {}
+    public record Response(List<SyncRecord> result, String error) {}
 
     private static List<SyncRecord> syncLast30Days(PelotonClient peloton, GarminClient garmin) {
         Instant thirtyDaysAgo = Instant.now().minus(Period.ofDays(30));

barista/src/main/java/barista/Handlers.java

@@ -1,18 +1,17 @@
 package barista;
 
+import static com.google.common.net.HttpHeaders.ACCESS_CONTROL_REQUEST_HEADERS;
+
 import com.google.common.base.Joiner;
 import io.undertow.server.HttpHandler;
 import io.undertow.server.HttpServerExchange;
 import io.undertow.util.HeaderValues;
 import io.undertow.util.Headers;
 import io.undertow.util.HttpString;
 import io.undertow.util.Methods;
-
 import java.util.Optional;
 import java.util.Set;
 
-import static com.google.common.net.HttpHeaders.ACCESS_CONTROL_REQUEST_HEADERS;
-
 public final class Handlers {
     private static final HttpString ACCESS_CONTROL_ALLOW_ORIGIN =
             new HttpString("Access-Control-Allow-Origin");
@@ -154,6 +153,7 @@ private <Request, Response> void openPost(
         } catch (Exception e) {
             exchange.setStatusCode(500);
             exchange.getResponseSender().send("Server Error");
+            e.printStackTrace();
         }
     }