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

Conversation

@JhontSouth
Copy link
Collaborator

#minor
Fixes # 6752

Description

This PR fixes the Dispose error avoiding the dispose of faulted tasks.

Specific Changes

  • Added if condition to dispose tasks that are not faulted.

Testing

The following image shows the bot responding correctly to other requests after an error has occurred.
image

@JhontSouth JhontSouth requested a review from a team as a code owner April 10, 2024 15:12
@JhontSouth JhontSouth added the Automation: No parity PR does not need to be applied to other languages. label Apr 10, 2024
@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 387573

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 4 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.01%) to 78.194%

Files with Coverage Reduction New Missed Lines %
/libraries/AdaptiveExpressions/LRUCache.cs 4 87.18%
Totals Coverage Status
Change from base Build 386500: -0.01%
Covered Lines: 26181
Relevant Lines: 33482

💛 - Coveralls

@BruceHaley
Copy link
Contributor

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll

@tracyboehrer tracyboehrer merged commit 2a488e4 into main Apr 10, 2024
@tracyboehrer tracyboehrer deleted the southworks/fix/dispose-error branch April 10, 2024 18:10
tracyboehrer pushed a commit that referenced this pull request Apr 15, 2024
…eption when an exception occurs in the bot (#6772)

* avoid faulted task to dispose

* remove nested if

* remove blank spaces
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.

5 participants