basecamp / thruster

MIT License
672 stars 16 forks source link

Add the conditional response with cache entity tag #14

Closed u5surf closed 2 months ago

u5surf commented 3 months ago

Nginx has the same mechanism can reply the 304 response using cache.

https://github.com/nginx/nginx/blob/e3207a17f084c9eb7905ca0f2cfdb1df088fd165/src/http/ngx_http_upstream.c#L6023-L-6043

https://github.com/nginx/nginx/blob/e3207a17f084c9eb7905ca0f2cfdb1df088fd165/src/http/modules/ngx_http_not_modified_filter_module.c#L86-L105

kevinmcconnell commented 2 months ago

Thanks @u5surf, this looks great 👍