vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #799 from mjibson/status-badge-cache

Browse source 
Set and use ETag headers on status badges
This commit is contained in:
Matt Jibson 2015-11-06 12:15:35 -05:00
commit 576fcf5325
1 changed files with 5 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

6
endpoints/web.py
View file

@ -399,7 +399,6 @@ def confirm_recovery():
@web.route('/repository/<path:repository>/status', methods=['GET']) @web.route('/repository/<path:repository>/status', methods=['GET'])
@parse_repository_name @parse_repository_name
@no_cache
@anon_protect @anon_protect
def build_status_badge(namespace, repository): def build_status_badge(namespace, repository):
token = request.args.get('token', None) token = request.args.get('token', None)
@ -423,8 +422,13 @@ def build_status_badge(namespace, repository):
else: else:
status_name = 'none' status_name = 'none'
if request.headers.get('If-None-Match') == status_name:
return Response(status=304)
response = make_response(STATUS_TAGS[status_name]) response = make_response(STATUS_TAGS[status_name])
response.content_type = 'image/svg+xml' response.content_type = 'image/svg+xml'
response.headers['Cache-Control'] = 'no-cache'
response.headers['ETag'] = status_name
return response return response