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

[BUG] Missing requestFields in DILoggerOptions #2592

Closed
KBroichhausen opened this issue Feb 12, 2024 · 2 comments · Fixed by #2593
Closed

[BUG] Missing requestFields in DILoggerOptions #2592

KBroichhausen opened this issue Feb 12, 2024 · 2 comments · Fixed by #2593
Assignees

Comments

@KBroichhausen
Copy link

Describe the bug

I tried to use requestFields as part of the logger object within the Configuration object passed via @Configuration decorator. As described here: https://tsed.io/docs/logger.html#configuration
Well, it's working but only with // @ts-ignore because the field is not defined in the DILoggerOptions.

To Reproduce

Just try to use something like

@Configuration({
  logger: {
    requestFields: ["headers"]
  },
        ....
}

Expected behavior

Typesafe compilation.

Code snippets

No response

Repository URL example

No response

OS

Windows 10

Node version

v18.7.1

Library version

v7.61.0

Additional context

No response

Romakita added a commit that referenced this issue Feb 17, 2024
…gger options from PlatformLogMiddleware

Closes: #2592
Romakita added a commit that referenced this issue Feb 17, 2024
…gger options from PlatformLogMiddleware

Closes: #2592
Romakita added a commit that referenced this issue Feb 17, 2024
…gger options from PlatformLogMiddleware

Closes: #2592
Romakita added a commit that referenced this issue Feb 17, 2024
…gger options from PlatformLogMiddleware

Closes: #2592
Copy link

🎉 Are you happy?

If you appreciated the support, know that it is free and is carried out on personal time ;)

A support, even a little bit makes a difference for me and continues to bring you answers!

github opencollective

@Romakita
Copy link
Collaborator

🎉 This issue has been resolved in version 7.61.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Romakita added a commit that referenced this issue Mar 2, 2024
…gger options from PlatformLogMiddleware

Closes: #2592
Romakita added a commit that referenced this issue Mar 3, 2024
…gger options from PlatformLogMiddleware

Closes: #2592
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
2 participants