fix: previously we forget to check if the value is nil (#8)

Author: spacewander

lib/resty/ngxvar/http.lua

@@ -163,6 +163,10 @@ end
 
 
 local function sum_upstream_num(s)
+    if type(s) ~= "string" then
+        return s
+    end
+
     local idx = str_find(s, " ", 1, true)
     if not idx then
         -- fast path

rockspec/lua-resty-ngxvar-0.5.2-0.rockspec

@@ -0,0 +1,20 @@
+package = "lua-resty-ngxvar"
+version = "0.5.2-0"
+source = {
+    url = "git://github.com/api7/lua-var-nginx-module",
+    tag = "v0.5.2"
+}
+description = {
+    summary = "Fetch nginx variable by FFI way for OpenResty which is faster",
+    homepage = "https://github.com/api7/lua-var-nginx-module",
+    license = "Apache License 2.0",
+    maintainer = "Yuansheng Wang <membphis@gmail.com>"
+}
+build = {
+    type = "builtin",
+    modules = {
+        ["resty.ngxvar"] = "lib/resty/ngxvar.lua",
+        ["resty.ngxvar.http"] = "lib/resty/ngxvar/http.lua",
+        ["resty.ngxvar.stream"] = "lib/resty/ngxvar/stream.lua",
+    }
+}

t/without_patch.t

@@ -109,3 +109,17 @@ number
 number
 102.023
 103.223
+
+
+
+=== TEST 4: check if value is nil before conversion
+--- config
+    location /t {
+        content_by_lua_block {
+            package.loaded.mock.upstream_header_time = nil
+            local var = require("resty.ngxvar")
+            ngx.say(var.fetch("upstream_header_time"))
+        }
+    }
+--- response_body
+nil