Skip to content

Handler config fix#77

Merged
axelsrz merged 1 commit intomainfrom
users/axsuarez/handler-config-fix
Jul 28, 2025
Merged

Handler config fix#77
axelsrz merged 1 commit intomainfrom
users/axsuarez/handler-config-fix

Conversation

@axelsrz
Copy link
Member

@axelsrz axelsrz commented Jul 28, 2025

This pull request includes fixes to logging and configuration handling in the microsoft-agents-hosting-core library. The most important changes address a typo in a logger call and improve the robustness of authorization handler initialization.

Logging Fix:

  • Corrected a typo in the logger call (ogger.errorlogger.error) in the _on_error method of agent_application.py. This ensures proper logging of errors.

Configuration Handling Improvements:

  • Fixed a typo in the variable name (handlers_congifhandlers_config) and improved the logic for initializing auth_handlers in authorization.py. The updated code ensures that auth_handlers is always initialized, even when no handlers are provided in the configuration.

@axelsrz axelsrz marked this pull request as ready for review July 28, 2025 22:50
@axelsrz axelsrz merged commit 97ffc42 into main Jul 28, 2025
5 checks passed
@axelsrz axelsrz deleted the users/axsuarez/handler-config-fix branch July 28, 2025 22:55
axelsrz added a commit that referenced this pull request Jul 28, 2025
axelsrz added a commit that referenced this pull request Jul 28, 2025
rodrigobr-msft pushed a commit that referenced this pull request Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments