plotly / falcon

Free, open-source SQL client for Windows and Mac 🦅

Home Page:https://plot.ly/free-sql-client-download/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Replace restify with express (and bunyan with winston)

n-riesco opened this issue · comments

At the moment, Falcon is stuck with restify@4 because restify@5 deprecated uncaughtException events. The latest version is restify@7.

Another issue that I've just come across is that restify pulls the logging module bunyan and the native module dtrace-provider as dependencies.

I'm opening this issue to get everyone's thoughts on replacing restify with express (and bunyan with winston). Please, could you add your 👍 / 👎 here?

@chriddyp @jackparmer @tarzzz @shannonlal @kndungu

Sounds good to me. @chriddyp had some reasons for using restify originally but I forget what they were.

I was using restify because it seemed "lighterweight" than express - we didn't need many of express's features like server-side templating, etc.
But from here on out, no preference from me, I'm good either way