Timeout configurable

[didierofrivia]

Closes #93

Example config:

extensions:
  limitador:
    type: "ratelimit",
    endpoint: "limitador-cluster",
    failureMode: "deny",
    timeout: "5s"
policies:
...

The timeout value must be string, and must be in the format of a duration. Default to 20ms

Examples of valid timeout values:

• 1ms parses as 1 millisecond
• 1.5ms parses as 1 millisecond and 500 microseconds
• 5s parses as 5 seconds

[adam-cattermole]

Nice job, this seems to be working well! I also verified by changing one of the integration test configs and could see the timeout in the grpc request change. Just needs a rebase I think

[alexsnaps]

lgtm, modulo that one comment on the negative durations. But this is internal again, so not a big deal, tho I'd create an issue if we don't fix it now in place. Thanks 🙏

[alexsnaps]

I think this could be an issue here, if I understand this right a "CelDuration" can be negative, a std can't if I'm not mistaken. So we expose it as if it could, then would panic if one would provide a negative one, which would be nonsensible in any case tho...

[didierofrivia]

you are correct, this would panic on a negative as OutOfRangeError(())