[Enhancement] Debug further on higher Limit for Cache TTL

Azure / data-api-builder

Data API builder provides modern REST and GraphQL endpoints to your Azure Databases and on-prem stores.

https://aka.ms/dab/docs

MIT License

947 stars 196 forks source link

[Enhancement] Debug further on higher Limit for Cache TTL #2443

Open sezal98 opened 3 weeks ago

sezal98 commented 3 weeks ago

We should have a higher limit on cache for TTL not only to keep a logical limit but also due to security reasons. DAB users shouldn't be allowed to use INT.Max as the upper limit which is the situation currently.

In this issue, we should explore with developers using DAB and get a logical conclusion for this limit.

JerryNixon commented 5 days ago

Can you add your reasoning for why Int.Max is a problem and why a higher limit is not?