-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use hof-logger rather than bespoke implementation #32
Comments
@joefitter hmm are there many options that can be overridden for logger? |
@joefitter has this been done? |
No, still to-do |
@joefitter can whoever does this please add an instance of the logger to the request object so it can be accessed by the controllers and middleware of any service that is using bootstrap? |
@easternbloc didn't you do this in churchill? |
@joefitter I started then got distracted |
@joefitter oh look a 🐝 |
lib/logger should be removed and replaced with hof-logger.
hof-logger allows defaults to be overridden so should we build this into the bootstrap options? Alternative is to be opinionated and use sensible defaults as per GRO, ROTM and BRP
The text was updated successfully, but these errors were encountered: