diff --git a/ivatar/views.py b/ivatar/views.py index 0a9abfe..d63d082 100644 --- a/ivatar/views.py +++ b/ivatar/views.py @@ -118,14 +118,13 @@ class AvatarImageView(TemplateView): if centry := caches["filesystem"].get(uri): # For DEBUG purpose only # print('Cached entry for %s' % uri) - response = HttpResponse( + return HttpResponse( centry["content"], content_type=centry["content_type"], status=centry["status"], reason=centry["reason"], charset=centry["charset"], ) - return response # In case no digest at all is provided, return to home page if "digest" not in kwargs: @@ -295,6 +294,8 @@ class AvatarImageView(TemplateView): imgformat = "jpeg" response = CachingHttpResponse(uri, data, content_type=f"image/{imgformat}") response["Cache-Control"] = "max-age=%i" % CACHE_IMAGES_MAX_AGE + # Remove Vary header for images since language doesn't matter + response["Vary"] = "" return response def _redirect_static_w_size(self, arg0, size): @@ -313,6 +314,8 @@ class AvatarImageView(TemplateView): data.seek(0) response = CachingHttpResponse(uri, data, content_type="image/png") response["Cache-Control"] = "max-age=%i" % CACHE_IMAGES_MAX_AGE + # Remove Vary header for images since language doesn't matter + response["Vary"] = "" return response def _return_cached_png(self, arg0, data, uri): @@ -408,6 +411,8 @@ class GravatarProxyView(View): data.read(), content_type=f"image/{file_format(img.format)}" ) response["Cache-Control"] = "max-age=%i" % CACHE_IMAGES_MAX_AGE + # Remove Vary header for images since language doesn't matter + response["Vary"] = "" return response except ValueError as exc: @@ -527,6 +532,8 @@ class BlueskyProxyView(View): data.read(), content_type=f"image/{file_format(format)}" ) response["Cache-Control"] = "max-age=%i" % CACHE_IMAGES_MAX_AGE + # Remove Vary header for images since language doesn't matter + response["Vary"] = "" return response except ValueError as exc: print(f"Value error: {exc}")