Skip to content

Limit connection earlier#244

Open
mbarnach wants to merge 3 commits intoKitura:masterfrom
mbarnach:close_ealy_connection_on_limit_issue_242
Open

Limit connection earlier#244
mbarnach wants to merge 3 commits intoKitura:masterfrom
mbarnach:close_ealy_connection_on_limit_issue_242

Conversation

@mbarnach
Copy link
Member

As pointed by @Davidde94 (#242), the connection should be closed earlier than on the end of the request.
This avoid dumping massive amount of data before checking the connection limit.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link

@Davidde94 Davidde94 left a comment

Choose a reason for hiding this comment

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

LGTM

@mbarnach mbarnach requested a review from dannys42 September 30, 2020 12:37
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