twilio / twilio-java

A Java library for communicating with the Twilio REST API and generating TwiML.
MIT License
485 stars 427 forks source link

chore: increased default number of total connections and connection p… #768

Closed sbansla closed 12 months ago

sbansla commented 12 months ago

Fixes

https://issues.corp.twilio.com/browse/DII-1175

When setMaxConnPerRoute(10) was set in HttpClientBuilder, it was not taking this value; instead, it was taking the default values set in PoolingHttpClientConnectionManager, which were 2 for defaultMaxPerRoute and 10 for maxTotal. Due to this setting, the client was experiencing excessive delays for each thread, subsequently leading to the failure of all requests.

To address the issue, the default number of total connections and connections per route were increased.

Checklist

If you have questions, please file a support ticket, or create a GitHub Issue in this repository.

sonarcloud[bot] commented 12 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.20.1) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17. Read more here