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.

Sign up for GitHub

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

Jump to bottom

Include logging scope properties in the log #209

Closed
mocsharp opened this issue Oct 13, 2022 · 1 comment · Fixed by #210
Closed

Include logging scope properties in the log #209

mocsharp opened this issue Oct 13, 2022 · 1 comment · Fixed by #210
Labels
enhancement Enhancement to existing feature
Milestone
0.3.3

Comments

@mocsharp
Copy link
Collaborator

Is your feature request related to a problem? Please describe.

The default JsonLayout does not include all the variables defined in log.BeginScope(...).

Describe the solution you'd like

Change the layout to:

<layout xsi:type="JsonLayout" includeAllProperties="true" IncludeMdlc="true">
@mocsharp mocsharp added the enhancement Enhancement to existing feature label Oct 13, 2022
@mocsharp mocsharp added this to the 0.3.3 milestone Oct 13, 2022
@mocsharp mocsharp mentioned this issue Oct 13, 2022
Merged
7 tasks
@github-actions
Copy link

🎉 This issue has been resolved in version 0.3.3 🎉

The release is available on:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Enhancement to existing feature
Projects
None yet
Milestone
0.3.3
Development

Successfully merging a pull request may close this issue.

Include scope properties Project-MONAI/monai-deploy-informatics-gateway
1 participant
@mocsharp