MINOR: cache: allow caching of OPTIONS request
Allow HAProxy to cache responses to OPTIONS HTTP requests.
This is useful in the use case of "Cross-Origin Resource Sharing" (cors)
to cache CORS responses from API servers.
Since HAProxy does not support Vary header for now, this would be only
useful for "access-control-allow-origin: *" use case.
diff --git a/src/cache.c b/src/cache.c
index 5b40623..0015326 100644
--- a/src/cache.c
+++ b/src/cache.c
@@ -560,8 +560,8 @@
if (!(txn->req.flags & HTTP_MSGF_VER_11))
goto out;
- /* cache only GET method */
- if (txn->meth != HTTP_METH_GET)
+ /* cache only GET or OPTIONS method */
+ if (txn->meth != HTTP_METH_GET && txn->meth != HTTP_METH_OPTIONS)
goto out;
/* cache key was not computed */
@@ -1058,6 +1058,9 @@
ctx.blk = NULL;
switch (txn->meth) {
+ case HTTP_METH_OPTIONS:
+ chunk_memcat(trash, "OPTIONS", 7);
+ break;
case HTTP_METH_HEAD:
case HTTP_METH_GET:
chunk_memcat(trash, "GET", 3);
@@ -1093,10 +1096,10 @@
struct cache_flt_conf *cconf = rule->arg.act.p[0];
struct cache *cache = cconf->c.cache;
- /* Ignore cache for HTTP/1.0 requests and for requests other than GET
- * and HEAD */
+ /* Ignore cache for HTTP/1.0 requests and for requests other than GET,
+ * HEAD and OPTIONS */
if (!(txn->req.flags & HTTP_MSGF_VER_11) ||
- (txn->meth != HTTP_METH_GET && txn->meth != HTTP_METH_HEAD))
+ (txn->meth != HTTP_METH_GET && txn->meth != HTTP_METH_HEAD && txn->meth != HTTP_METH_OPTIONS))
txn->flags |= TX_CACHE_IGNORE;
http_check_request_for_cacheability(s, &s->req);