global: add Cache-Control header

inveniosoftware / invenio-records-rest

Invenio records REST API module.

https://invenio-records-rest.readthedocs.io

MIT License

4 stars 63 forks source link

global: add Cache-Control header #302

Closed ntarocco closed 4 years ago

ntarocco commented 4 years ago

adds Cache-Control:'no-cache' header to 200 responses to ensure that browsers will not cache responses client side
fixes isort to support v5

ntarocco commented 4 years ago

Do we want to depend on invenio-rest specific min version, after the release of https://github.com/inveniosoftware/invenio-rest/pull/114? Personally, I would say no... but there is a test failing because the current version of invenio-rest is not returning the extra header Cache-Control in case of 304 (same content)...