Skip to content

Fix missing dispose on rate limiting - #76784

Merged
BrennanConroy merged 1 commit into
dotnet:mainfrom
AliKhalili:issue-76783-ratelimit-disposed-obsolete-request
Nov 2, 2022
Merged

Fix missing dispose on rate limiting#76784
BrennanConroy merged 1 commit into
dotnet:mainfrom
AliKhalili:issue-76783-ratelimit-disposed-obsolete-request

Conversation

@AliKhalili

Copy link
Copy Markdown
Contributor

Proposed fix for #76783

@ghostghost added area-System.Threading community-contribution Indicates that the PR has been added by a community member labels Oct 8, 2022
@ghost

ghost commented Oct 8, 2022

Copy link
Copy Markdown

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

Issue Details

Proposed fix for #76783

Author:AliKhalili
Assignees:-
Labels:

area-System.Threading

Milestone:-

@AliKhalili
AliKhalili marked this pull request as ready for review October 8, 2022 19:56
@AliKhaliliAliKhalili changed the title Fixe missing dispose on rate limitingFix missing dispose on rate limitingOct 12, 2022

@BrennanConroyBrennanConroy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@BrennanConroy

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@BrennanConroy

Copy link
Copy Markdown
Member

/azp run runtime

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@BrennanConroy
BrennanConroy merged commit 37bd664 into dotnet:mainNov 2, 2022
@BrennanConroyBrennanConroy added this to the 8.0.0 milestone Nov 2, 2022
@AliKhalili
AliKhalili deleted the issue-76783-ratelimit-disposed-obsolete-request branch November 2, 2022 20:27
@ghostghost locked as resolved and limited conversation to collaborators Dec 3, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Threadingcommunity-contributionIndicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@AliKhalili@BrennanConroy