optimize: support to cache the current request object, make it faster.

Author: membphis

lib/resty/ngxvar.lua

@@ -18,23 +18,12 @@ int ngx_http_lua_var_ffi_remote_addr(ngx_http_request_t *r,
 
 
 local var_patched = pcall(function() return C.ngx_http_lua_var_ffi_test() end)
-local _M = {
-    _version = 0.1,
+local vars = {
     method = ngx.req.get_method,
 }
 
 
-function _M._request()
-    local r = get_request()
-    if not r then
-        return false, "no request found"
-    end
-
-    return r
-end
-
-
-function _M.uri(r)
+function vars.uri(r)
     r = r or get_request()
     if not r then
         return false, "no request found"
@@ -45,7 +34,7 @@ function _M.uri(r)
 end
 
 
-function _M.host(r)
+function vars.host(r)
     r = r or get_request()
     if not r then
         return false, "no request found"
@@ -56,12 +45,12 @@ function _M.host(r)
 end
 
 
-function _M.status()
+function vars.status()
     return ngx.status
 end
 
 
-function _M.remote_addr(r)
+function vars.remote_addr(r)
     r = r or get_request()
     if not r then
         return false, "no request found"
@@ -72,12 +61,30 @@ function _M.remote_addr(r)
 end
 
 
-return function (name, request)
-    local method = _M[name]
+local _M = {
+    _version = 0.1,
+}
+
+
+function _M.request()
+    local r = get_request()
+    if not r then
+        return false, "no request found"
+    end
+
+    return r
+end
+
+
+function _M.fetch(name, request)
+    local method = vars[name]
 
     if not var_patched or not method then
         return ngx_var[name]
     end
 
     return method(request)
 end
+
+
+return _M

t/sanity.t

@@ -23,7 +23,7 @@ __DATA__
     location /t {
         content_by_lua_block {
             local var = require("resty.ngxvar")
-            ngx.say(var("uri"))
+            ngx.say(var.fetch("uri"))
         }
     }
 --- request
@@ -41,7 +41,7 @@ GET /t/test/bar
     location /t {
         content_by_lua_block {
             local var = require("resty.ngxvar")
-            ngx.say(var("host"))
+            ngx.say(var.fetch("host"))
         }
     }
 --- request
@@ -64,7 +64,8 @@ foo.com
         }
         log_by_lua_block {
             local var = require("resty.ngxvar")
-            ngx.log(ngx.ERR, var("status"), " type: ", type(var("status")))
+            ngx.log(ngx.ERR, var.fetch("status"),
+                    " type: ", type(var.fetch("status")))
         }
     }
 --- request
@@ -81,8 +82,8 @@ GET /t
     location /t {
         content_by_lua_block {
             local var = require("resty.ngxvar")
-            local req = var("_request")
-            ngx.say(var("host", req))
+            local req = var.fetch("_request")
+            ngx.say(var.fetch("host", req))
         }
     }
 --- request
@@ -102,7 +103,7 @@ foo.com
     location /t {
         content_by_lua_block {
             local var = require("resty.ngxvar")
-            ngx.say(var("remote_addr"))
+            ngx.say(var.fetch("remote_addr"))
         }
     }
 --- request