Allow /api/scratch/ABCDE to be cached for 1s (#1773)

* Allow /api/scratch/ABCDE to be cached for 1s

* Update user status on public endpoints

* Don't prefetch /new, /platform and /preset on homepage

Author: mkst

backend/coreapp/middleware.py

@@ -49,6 +49,8 @@ def is_public_request(req: Request) -> bool:
         ("GET", "/api/preset"),
         ("GET", "/api/scratch-count$"),
         ("GET", "/api/scratch/[A-Za-z0-9]+/compile$"),
+        ("GET", "/api/scratch/[A-Za-z0-9]+/export$"),
+        ("GET", "/api/scratch/[A-Za-z0-9]+/family$"),
         ("GET", "/api/scratch/[A-Za-z0-9]+$"),
         ("GET", "/api/scratch$"),
         ("GET", "/api/search$"),
@@ -87,11 +89,6 @@ def middleware(request: Request) -> Response:
             request.profile = Profile()
             return get_response(request)
 
-        # Avoid creating persistent for public endpoints
-        if is_public_request(request):
-            request.profile = Profile()
-            return get_response(request)
-
         profile = None
 
         # Try user-linked profile
@@ -109,6 +106,11 @@ def middleware(request: Request) -> Response:
                 if profile and profile.user and request.user.is_anonymous:
                     request.user = profile.user
 
+        # Avoid creating persistent for public endpoints
+        if not profile and is_public_request(request):
+            request.profile = Profile()
+            return get_response(request)
+
         # Create new profile if none found
         if not profile:
             profile = Profile(

backend/coreapp/views/scratch.py

@@ -268,6 +268,7 @@ class ScratchPagination(CursorPagination):
 
 
 @method_decorator(globally_cacheable(max_age=5, stale_while_revalidate=1), name="list")
+@method_decorator(globally_cacheable(max_age=1), name="retrieve")
 class ScratchViewSet(
     mixins.CreateModelMixin,
     mixins.DestroyModelMixin,

frontend/src/app/(navfooter)/WelcomeInfo.tsx

@@ -46,7 +46,7 @@ export default async function WelcomeInfo() {
                     {SITE_DESCRIPTION}
                 </p>
                 <div className="flex flex-col items-center justify-center gap-2 md:flex-row">
-                    <Link href="/new">
+                    <Link href="/new" prefetch={false}>
                         <Button primary>
                             Start decomping
                             <ArrowRightIcon />
@@ -56,11 +56,15 @@ export default async function WelcomeInfo() {
                 </div>
                 <p className="mx-auto my-6 w-full max-w-screen-sm text-gray-11 text-sm leading-tight">
                     Alternatively, check out existing scratches filtered by{" "}
-                    <Link className="font-bold" href="/platform">
+                    <Link
+                        className="font-bold"
+                        href="/platform"
+                        prefetch={false}
+                    >
                         platform
                     </Link>{" "}
                     or{" "}
-                    <Link className="font-bold" href="/preset">
+                    <Link className="font-bold" href="/preset" prefetch={false}>
                         preset
                     </Link>
                     .