resurfaceio / logger-nodejs

Log API calls with Node.js

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Missing null checks

RobDickinson opened this issue · comments

We should be checking for null in a lot of places where we're checking for undefined.

  • passing {url: null} to BaseLogger constructor
  • passing null param to request_body or response_body to HttpLogger.format

https://www.davidbcalhoun.com/2011/checking-for-undefined-null-and-empty-variables-in-javascript/