Skip to content
This repository has been archived by the owner on May 5, 2022. It is now read-only.

Enable verbosity config #25

Open
a-roberts opened this issue Jan 25, 2018 · 0 comments
Open

Enable verbosity config #25

a-roberts opened this issue Jan 25, 2018 · 0 comments

Comments

@a-roberts
Copy link
Contributor

It'd be useful to have, in appmetrics-zipkin.properties for example, and via a method call, to be able to configure how noisy the module behaves.

Either we use a standard Node.js logging library (so whatever the log4j equivalent is) or do it ourselves with custom levels - for now I'm really only looking for silent and normal.

I expect, in appmetrics-zipkin.propeties, to have:
host=...
port=...
logging=silent for example.

When the silent option is provided, i want to have zero errors reported (so when requests can't reach a Zipkin server because it's not yet configured or available for example).

With normal I'd want the behaviour as it is now. Anything extra (e.g. providing trace data) should be considered as part of #24 IMO.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant