L1-node icon indicating copy to clipboard operation
L1-node copied to clipboard

Remove express

Open DiegoRBaquero opened this issue 3 years ago • 3 comments

L1 doesn't need a full web framework. Express should be removed.

DiegoRBaquero avatar Jul 26 '22 13:07 DiegoRBaquero

I personally like using little wrappers like https://github.com/electron/update.electronjs.org/blob/f0d89fe197ed13b63fbe58af96492eab1b4c21cc/index.js#L42-L60, which help with error responses and request logging

juliangruber avatar Jul 26 '22 16:07 juliangruber

Instead of going frameworkless, @bajtos suggested looking at fastify

juliangruber avatar Aug 22 '22 09:08 juliangruber

Should we not remove express, we must secure the current async route handlers, which aren't supported natively by the framework. @guanzo suggested using https://www.npmjs.com/package/express-async-errors in this case.

juliangruber avatar Aug 22 '22 09:08 juliangruber