cjson.decode should be wrapped in a pcall

[ddragosd]

This is where the error occurs.

val = cjson.decode(val_string)

It seems to be an edge case and we need to add a try/catch for this scenario. I’m assuming this is happening when a rule expires and we have an incoming request in that millisecond and the shared dictionary returns an empty string for a rule causing the cjson lib to complain ?... one possiblity

We should wrap the decode into a pcall as follows:

local ok, val = pcall(cjson.decode,val_string)
if not ok then
   -- continue
end

[cristianconstantin]

I've investigated and found that indeed the empty value is added to the cache in this line:

api-gateway-request-tracking/src/lua/api-gateway/tracking/RequestTrackingManager.lua

Line 93 in 55b9e12

dict:set(rule.id .. " " .. rule.format, "", 0.001, 0)

.

 dict:set(rule.id .. " " .. rule.format, "", 0.001, 0)

Wouldn't it be better to just remove the key from the shared dictionary?

[ddragosd]

👍 @cristianconstantin
There's a delete api now.