.Net: Logging improvements in OpenAPI plugins - Githubissues

microsoft / semantic-kernel

Integrate cutting-edge LLM technology quickly and easily into your apps

https://aka.ms/semantic-kernel

MIT License

22.07k stars 3.29k forks source link

.Net: Logging improvements in OpenAPI plugins #9692

Closed dmytrostruk closed 1 week ago

dmytrostruk commented 1 week ago

Motivation and Context

Resolves: https://github.com/microsoft/semantic-kernel/issues/6887

This PR contains logging improvements for OpenAPI plugin functionality:

Added usage of ILoggerFactory/ILogger in places where null instance was used.
Improved logging message parameter names in existing logging logic.
Added more logging in case of exceptions.
Added an example how to enable logging in OpenAPI plugin functionality.

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile: