anvilresearch / connect

A modern authorization server built to authenticate your users and protect your APIs
http://anvil.io
MIT License
361 stars 85 forks source link

fix(logger) default logger.js location #347

Closed hedleysmith closed 7 years ago

hedleysmith commented 7 years ago

Currently a custom logger.js file is expected to be found in the anvil-connect node package root - if a logger.js file is added there it's deleted every time node_modules is reinstalled.

As discussed in the original commit here: https://github.com/anvilresearch/connect/commit/68ba379e4355ae70b36ea2d1a0f1ddf18b90a562 this PR changes the behaviour to look relative to where anvil is being run from.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 79.743% when pulling 6b18c7c2b2d4effa29f59e26d691e1d9ebcfccd6 on hedleysmith:logger-js-path into b31264d76fd71158ab7b54db576a9cf8dafc0918 on anvilresearch:master.

christiansmith commented 7 years ago

@hedleysmith that's a great catch. Thanks!