diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/change/Files.java b/gerrit-server/src/main/java/com/google/gerrit/server/change/Files.java index 54f4f292f8..89ec948ad5 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/change/Files.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/change/Files.java @@ -16,9 +16,11 @@ package com.google.gerrit.server.change; import com.google.gerrit.extensions.registration.DynamicMap; import com.google.gerrit.extensions.restapi.AuthException; +import com.google.gerrit.extensions.restapi.CacheControl; import com.google.gerrit.extensions.restapi.ChildCollection; import com.google.gerrit.extensions.restapi.IdString; import com.google.gerrit.extensions.restapi.ResourceNotFoundException; +import com.google.gerrit.extensions.restapi.Response; import com.google.gerrit.extensions.restapi.RestReadView; import com.google.gerrit.extensions.restapi.RestView; import com.google.gerrit.reviewdb.client.PatchSet; @@ -29,6 +31,8 @@ import com.google.inject.Provider; import org.kohsuke.args4j.Option; +import java.util.concurrent.TimeUnit; + class Files implements ChildCollection { private final DynamicMap> views; private final FileInfoJson fileInfoJson; @@ -73,8 +77,11 @@ class Files implements ChildCollection { resource.getChangeResource(), IdString.fromDecoded(base)); basePatchSet = baseResource.getPatchSet(); } - return fileInfoJson.toFileInfoMap( - resource.getChange(), resource.getPatchSet(), basePatchSet); + return Response.ok(fileInfoJson.toFileInfoMap( + resource.getChange(), + resource.getPatchSet(), + basePatchSet)) + .caching(CacheControl.PRIVATE(7, TimeUnit.DAYS)); } } }