Skip to content
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

Add conditional logging #51

Closed
ASafaeirad opened this issue Jul 4, 2018 · 5 comments
Closed

Add conditional logging #51

ASafaeirad opened this issue Jul 4, 2018 · 5 comments
Labels
feature request Request for a new feature

Comments

@ASafaeirad
Copy link

It would be nice to add a property to the custom options that can control logging on a certain condition

Example

const option = {
  stream: process.stdout,
  debug: {
     //...
     condition: process.env.LOG_LEVEL === 'info'
  }
};

proccess.env.LOG_LELEV = 'info';
logger.debug('Some debug info');
> Some debug info

proccess.env.LOG_LEVEL = 'error';
logger.debug('Some debug info');
>
@ivokund
Copy link

ivokund commented Jul 11, 2018

This would be really useful, I came looking for just something like that.

I'd love an option to specify different verbosity levels and run my script with different loggers enabled. For example

node index.js -v # enables all loggers, except debug/trace
node index.js -vv # enables all loggers

In code you could do something like this:

// 2 could be the default level. 0 = no logging
logger.setVerbosity(2);
// or set it in options
const option = {
  verbosity: 2
};
// and specify verbosity requirement for each logger to be activated
const option = {
  debug: {
     verbosity: 2,
  }
};

@klaudiosinani klaudiosinani added the feature request Request for a new feature label Aug 5, 2018
@klaudiosinani
Copy link
Owner

klaudiosinani commented Aug 5, 2018

Yep this is definitely needed, the feature will land on the upcoming release : )

@ericksepulveda
Copy link

+1 to this feature request

@felixfbecker
Copy link

I can't get this to work to override through signale.config({ logLevel: ... })

@Oxicode
Copy link

Oxicode commented Feb 10, 2020

Yep this is definitely needed, the feature will land on the upcoming release : )

Please 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Request for a new feature
Projects
None yet
Development

No branches or pull requests

6 participants