From 53fb93b232208de7144e13990bb2ce0cb5467899 Mon Sep 17 00:00:00 2001 From: silverwind Date: Mon, 6 Sep 2021 18:37:22 +0200 Subject: [PATCH] Add Cache-Control to avatar redirects This does seem to do the trick to make the Avatar redirects cachable in Chrome. In Firefox, it does not seem to work, thought and I found no way to suppress the requests to the original URLs, I even tried setting an Etag to no avail. Related discussion in https://github.com/go-gitea/gitea/issues/16964. --- routers/web/user/avatar.go | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/routers/web/user/avatar.go b/routers/web/user/avatar.go index 8a5a36e070a91..baa73f8904183 100644 --- a/routers/web/user/avatar.go +++ b/routers/web/user/avatar.go @@ -13,10 +13,16 @@ import ( "code.gitea.io/gitea/models" "code.gitea.io/gitea/modules/context" + "code.gitea.io/gitea/modules/httpcache" "code.gitea.io/gitea/modules/log" "code.gitea.io/gitea/modules/setting" ) +func doRedirect(ctx *context.Context, location string) { + ctx.Resp.Header().Set("Cache-Control", httpcache.GetCacheControl()) + ctx.Redirect(location) +} + // Avatar redirect browser to user avatar of requested size func Avatar(ctx *context.Context) { userName := ctx.Params(":username") @@ -43,7 +49,7 @@ func Avatar(ctx *context.Context) { user = models.NewGhostUser() } - ctx.Redirect(user.RealSizedAvatarLink(size)) + doRedirect(ctx, user.RealSizedAvatarLink(size)) } // AvatarByEmailHash redirects the browser to the appropriate Avatar link @@ -63,7 +69,7 @@ func AvatarByEmailHash(ctx *context.Context) { return } if len(email) == 0 { - ctx.Redirect(models.DefaultAvatarLink()) + doRedirect(ctx, models.DefaultAvatarLink()) return } size := ctx.FormInt("size") @@ -94,5 +100,5 @@ func AvatarByEmailHash(ctx *context.Context) { } } - ctx.Redirect(models.MakeFinalAvatarURL(avatarURL, size)) + doRedirect(ctx, models.MakeFinalAvatarURL(avatarURL, size)) }