node-restify icon indicating copy to clipboard operation
node-restify copied to clipboard

Allow overriding log in `plugins/audit`

Open juneidy opened this issue 1 year ago • 0 comments

  • [ x] Used appropriate template for the issue type
  • [ x] Searched both open and closed issues for duplicates of this issue
  • [ x] Title adequately and concisely reflects the feature or the bug

Feature Request

Use Case

So the change in 528ecbcec5d70c458749bdd4c4cc3f9e06ab69a2 means that whatever log I passed into auditLogger is not prioritised over the log from createServer.

I would imagine that whatever new log passed to auditLogger should take precedence.

Example API

const server = restify.createServer({ log: loggerA , ... });
server.on('after', restify.plugins.auditLogger({ log: loggerB, ... }));

I would expect loggerB would be used instead of loggerA.

Are you willing and able to implement this?

I can quickly do the fix, but not sure about the tests.

juneidy avatar Mar 15 '23 06:03 juneidy