Skip to content

perf(api): set cache headers for responses#184

Open
huazhuang80-star wants to merge 1 commit into
ChainForgee:mainfrom
huazhuang80-star:api-cache-control-headers
Open

perf(api): set cache headers for responses#184
huazhuang80-star wants to merge 1 commit into
ChainForgee:mainfrom
huazhuang80-star:api-cache-control-headers

Conversation

@huazhuang80-star

Copy link
Copy Markdown

Summary

  • enable weak ETags on the Express adapter
  • add Cache-Control headers for API responses
  • avoid caching authenticated or state-changing requests by marking them no-store

Fixes #129

Testing

  • git diff --check
  • Not run:
    pm --prefix app/backend run lint:check (node_modules is not installed in this checkout)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[LOW] No cache headers set on API responses

1 participant