Access log level in logMethod hook

[gzaripov]

I want to collect logs number statistics and I want also to categorize them by levels.
For now I cant get level from logMethod hook, so I cant collect info about method types, lets add it to logMethod hook:

const hooks = {
  logMethod (inputArgs, method, { level }) {
    metrics.registerEvent(`log_${level}`);
    return method.apply(this, inputArgs)
  }
}

[mcollina]

It is not needed as you can get it with https://github.com/pinojs/pino/blob/83d80f63888aa0b68479d460706644c58257131c/docs/api.md#levels.

logger.levels.values[method]

[gzaripov]

@mcollina could you provide exact solution? I try to call this.levels.values[method] inside function and I get undefined.
Here is reproduction https://codesandbox.io/s/funny-shannon-e8elg?file=/src/start.js

To see logs just make some change, i.e. add enter and press cmd+s, it will re-run server and logs come up in terminal window

[mcollina]

Actually I was wrong, that's not feasible right now.

What's your use case exactly?

[jsumners]

Support would need to be added at:

pino/lib/tools.js

Lines 30 to 35 in 95856d1

	function genLog (level, hook) {
	if (!hook) return LOG
	return function hookWrappedLog (...args) {
	hook.call(this, args, LOG)
	}

[gzaripov]

What's your use case exactly?

I want to provide more info about what kind of logs we are logging in node server. It will help us to track new problems when error logs count was increased. For now I can track only total amount of logs.

[gzaripov]

I suggest this solution:

function genLog (level, hook) {
  if (!hook) return LOG

  return function hookWrappedLog (...args) {
    hook.call(this, args, LOG, { level })
  }

[mcollina]

Please avoid creating a new object, e.g. level not { level }.

[mojavelinux]

FYI, this appears to have been resolved but never closed.

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.