search

jaysylvester / citizen

Node.js MVC web application framework. Includes routing, serving, caching, session management, and other helpful tools.
MIT License
100 stars 7 forks source link

Wrap all cache calls that can throw errors in try/catch blocks #90

Closed jaysylvester closed 3 years ago

jaysylvester commented 3 years ago

Some cache methods throw errors that crash the app.

jaysylvester commented 3 years ago

Rather than throw errors intentionally, I decided it's better to get rid of the overwrite option that even I've never used personally and just overwrite existing caches.

All the remaining errors should only occur in a dev environment because they reveal obvious bugs in the app's code, not citizen.