From 2a916a336deb09b8949975b94d57eebeca07bbbf Mon Sep 17 00:00:00 2001
From: Liam <liam@fascinated.cc>
Date: Sat, 6 Apr 2024 19:16:54 +0100
Subject: [PATCH] add cache control to the player endpoint

---
 .../cc/fascinated/api/controller/PlayerController.java    | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/main/java/cc/fascinated/api/controller/PlayerController.java b/src/main/java/cc/fascinated/api/controller/PlayerController.java
index a7c79cd..331b36f 100644
--- a/src/main/java/cc/fascinated/api/controller/PlayerController.java
+++ b/src/main/java/cc/fascinated/api/controller/PlayerController.java
@@ -20,6 +20,7 @@ import java.util.concurrent.TimeUnit;
 @RequestMapping(value = "/")
 public class PlayerController {
 
+    private final CacheControl cacheControl = CacheControl.maxAge(1, TimeUnit.HOURS).cachePublic();
     @NonNull private final SkinPart defaultHead = Objects.requireNonNull(Skin.getDefaultHead(), "Default head is null");
     private final PlayerManagerService playerManagerService;
 
@@ -34,7 +35,10 @@ public class PlayerController {
         if (player == null) {
             return new ResponseEntity<>(Map.of("error", "Player not found"), HttpStatus.NOT_FOUND);
         }
-        return ResponseEntity.ok(player);
+        return ResponseEntity.ok()
+                .cacheControl(cacheControl)
+                .body(player);
+
     }
 
     @GetMapping(value = "/avatar/{id}")
@@ -49,7 +53,7 @@ public class PlayerController {
             headBytes = head.getPartData();
         }
         return ResponseEntity.ok()
-                .cacheControl(CacheControl.maxAge(1, TimeUnit.HOURS).cachePublic())
+                .cacheControl(cacheControl)
                 .contentType(MediaType.IMAGE_PNG)
                 .body(headBytes);
     }