Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

Logging UnauthorizedAccessException in BotFrameworkHttpAdapter#6784

Merged
tracyboehrer merged 1 commit into
mainfrom
tracyboehrer/adapter-auth-logging
May 7, 2024
Merged

Logging UnauthorizedAccessException in BotFrameworkHttpAdapter#6784
tracyboehrer merged 1 commit into
mainfrom
tracyboehrer/adapter-auth-logging

Conversation

@tracyboehrer
Copy link
Copy Markdown
Member

#minor

Makes it easier to see auth errors due to misconfiguration.

@tracyboehrer tracyboehrer added the Automation: No parity PR does not need to be applied to other languages. label May 7, 2024
@tracyboehrer tracyboehrer requested a review from a team as a code owner May 7, 2024 17:59
@coveralls
Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 388816

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 35 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.002%) to 78.163%

Files with Coverage Reduction New Missed Lines %
/libraries/AdaptiveExpressions/LRUCache.cs 4 87.18%
/libraries/integration/Microsoft.Bot.Builder.Integration.AspNet.Core/BotFrameworkHttpAdapter.cs 31 50.93%
Totals Coverage Status
Change from base Build 388810: -0.002%
Covered Lines: 26187
Relevant Lines: 33503

💛 - Coveralls

@tracyboehrer tracyboehrer merged commit 673dbca into main May 7, 2024
@tracyboehrer tracyboehrer deleted the tracyboehrer/adapter-auth-logging branch May 7, 2024 18:22
This was referenced May 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Automation: No parity PR does not need to be applied to other languages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants