minilog icon indicating copy to clipboard operation
minilog copied to clipboard

suggest.deny filter change no longer matches docs and breaks deny()

Open nevf opened this issue 7 years ago • 1 comments

The commit https://github.com/mixu/minilog/commit/bcf518bec2ba84a430dd7b23caea32d6b1a5d01c changed the blacklist deny test from < to <= which disagrees with the docs http://mixu.net/minilog/filter.html

This configures the default filter to hide logs where the level is < 'warn'. 

I don't understand why this change was made as you can no longer use:

Minilog.suggest.deny(/mymodule\/.*/, 'debug');

to filter out debug items etc.

nevf avatar Jun 11 '18 01:06 nevf

The documentation is out of date but the ability to deny debug logs should still work.

Currently: filter.deny() will deny the log level specified and below. filter.allow() will allow the log level specified and above.

The code given should work, I've tested it locally and able to deny debug level logs. Can you post an example of the issue?

roark avatar Feb 07 '19 07:02 roark