Cache headers added to /maven endpoint

quarkusio / registry.quarkus.io

Quarkus Extension Registry application

https://registry.quarkus.io

Apache License 2.0

10 stars 11 forks source link

Cache headers added to /maven endpoint #12

Closed phillip-kruger closed 3 years ago

phillip-kruger commented 3 years ago

This PR is mainly to add the cache related headers to the response for /maven endpoint.

This is the Etag that is a UUID based on the response content and a last modified based on the cache date.

This also do (separate commits):

Moved clear cache to admin
Only cache 200 responses
Changed content to Object (rather than casting to String)
Clean up some unused imports
Added content type header to pom response

phillip-kruger commented 3 years ago

@Sanne - as per email, you might be interested in this. Specifically the headers part in this commit: https://github.com/quarkusio/registry.quarkus.io/pull/12/commits/18a79bd3835ac7346b033eb0dd7715a1f28e9f31

gastaldi commented 3 years ago

@phillip-kruger can you rebase your PR on latest main?

UPDATE: Already did, nvm ;)

Sanne commented 3 years ago

@gastaldi no, thanks no need. You guys seem to know this stuff way better than me, I just wanted to alert you on difficulties with CDNs but it seems you know :)