Skip to content

Cache routes only when caching enabled#849

Closed
tsteur wants to merge 1 commit into
livefrom
cache_route
Closed

Cache routes only when caching enabled#849
tsteur wants to merge 1 commit into
livefrom
cache_route

Conversation

@tsteur

@tsteur tsteur commented Oct 30, 2025

Copy link
Copy Markdown
Member

I pulled the latest changes from Git recently and whenever I open the changelog page locally, it would render the trac page. After a quick debug session noticed that the routes are cached despite caching disabled. I assume that this cache shouldn't be set unless caching is actually enabled. Might also give a performance update on production

Review

I pulled the latest changes from Git recently and whenever I open the changelog page locally, it would render the trac page. After a quick debug session noticed that the routes are cached despite caching disabled. I assume that this cache shouldn't be set unless caching is actually enabled. Might also give a performance update on production
@michalkleiner

Copy link
Copy Markdown
Contributor

There is already a PR for this with a discussion #842

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

Development

Successfully merging this pull request may close these issues.

2 participants