Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Per vhost cache #2366

Open
const-t opened this issue Mar 18, 2025 · 0 comments
Open

Per vhost cache #2366

const-t opened this issue Mar 18, 2025 · 0 comments
Labels
bug cache good to start Start form this tasks if you're new in Tempesta FW
Milestone

Comments

@const-t
Copy link
Contributor

const-t commented Mar 18, 2025

Cache key consist of uri + host header, however may be the case when http_chains is configured to route request to other vhost regardless host header. Therefore, after routing Tempesta will try to find/build cache record using the key that belongs to another vhost(specified in host header).

Example: curl -H"Connection: close" http://app1

vhost app1 {
...
}

vhost app2 {
...
}

http_chains {
hdr "Connection" == "keep-alive" > app1;
-> app2;
}

In this case, although we requesting app1 request will be forwarded to app2, but cache record will be stored using key / app1, however expected key is / app2.

To fix this issue we must take uri + vhost name to build the cache key instead of uri + host header.

Testing

Functional test must be implemented.

@const-t const-t added bug cache good to start Start form this tasks if you're new in Tempesta FW labels Mar 18, 2025
@const-t const-t added this to the 0.9 - LA milestone Mar 18, 2025
@krizhanovsky krizhanovsky modified the milestones: 0.9 - LA, 1.0 - GA Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cache good to start Start form this tasks if you're new in Tempesta FW
Projects
None yet
Development

No branches or pull requests

2 participants