Merge pull request Kong#296 from Mashape/fix/schema-strip-strings

[fix][schemas] strip string values

Former-commit-id: 0e5b1b6506abd530b5a3ded139cf82eaae19b4a3

kong/dao/schemas_validation.lua

@@ -57,6 +57,7 @@ function _M.validate(t, schema, is_update)
 
       -- ALIASES: number, timestamp, boolean and array can be passed as strings and will be converted
       if type(t[column]) == "string" then
+        t[column] = stringy.strip(t[column])
         if v.type == "number" or v .type == constants.DATABASE_TYPES.TIMESTAMP then
           t[column] = tonumber(t[column])
           is_valid_type = t[column] ~= nil
@@ -66,6 +67,9 @@ function _M.validate(t, schema, is_update)
           t[column] = bool == "true"
         elseif v.type == "array" then
           t[column] = stringy.split(t[column], ",")
+          for arr_k, arr_v in ipairs(t[column]) do
+            t[column][arr_k] = stringy.strip(arr_v)
+          end
           is_valid_type = validate_type(v.type, t[column])
         else -- if string
           is_valid_type = validate_type(v.type, t[column])

spec/unit/schemas_spec.lua

@@ -184,7 +184,8 @@ describe("Schemas", function()
       it("should alias a string to `array`", function()
         local s = { array = { type = "array" } }
 
-        local values = { array = "hello,world" }
+        -- It should also strip the resulting strings
+        local values = { array = "hello, world" }
 
         local valid, err = validate(values, s)
         assert.True(valid)