[FEATURE REQUEST] Correct stack trace of a guard error in request errored pino-http's log

[artursudnik]

The problem I am facing is similar to those described in this feature request. Despite after using LoggerErrorInterceptor I am getting the correct stack trace of errors thrown from controllers or services when an error is thrown from a guard, it behaves like LoggerErrorInterceptor was not used.

Describe the solution you'd like
I am not sure if it is a bug or if additional functionality to be added. Probably this problem is caused by Interceptor not being called at all when application flow is broken on a guard level.

[iamolegga]

can you provide minimal example repo with logs?

[artursudnik]

@iamolegga sure, I'll isolate this

[artursudnik]

@iamolegga
I apologize for the delay.
Here is this problem reproduced: https://github.com/artursudnik/nestjs-pino-issue.

[mattvgm]

I can confirm this happens to me either. Have you came up with a workaround to it?

[artursudnik]

No, unfortunately, I have not.

[iamolegga]

Does it work properly with the original built-in logger?

[artursudnik]

@iamolegga I need to double-check this. I guess yes though.

[eyalch]

Considering NestJS's request lifecycle, should this logic be in an interceptor? Wouldn't an exception filter be a better fit for this?

The same issue is happening for me. All errors except ones from guards are logged just fine.

[iamolegga]

Wouldn't an exception filter be a better fit for this?

agree, we need to change this class to a filter and add tests to verify that it correctly catches and logs errors from different layers

[magbeans]

Same to me, but it only happens when i try to execute test scripts using jest+supertest.
I am not sure how to fix it.
@artursudnik Did you find out the solution?

[artursudnik]

@happy-ruby

@artursudnik Did you find out the solution?

No, I have not.

[magbeans]

Okay thanks, what i fixed is that i switched logger.warn to logger.errror in exception cases.
Then i could catch out the issue logs.
That may be an alternative solution.