search

dustinblackman / Championify

Import recent item sets from popular aggregators like Champion.gg in to League of Legends to use within game! No hassle.
MIT License
890 stars 157 forks source link

Feature/limit concurrent requests #342

Closed gerriet-hinrichs closed 6 years ago

gerriet-hinrichs commented 6 years ago

Limit concurrent requests to prevent resource blocking. This is required since computers often limit the amount of TCP connections that are established concurrently. The result of this behaviour is further discussed on issue 339. Implemented limits are based on limitations in modern webbrowsers (e.g. firefox, chrome) and are set to max 17 concurrent connections in total and max 6 concurrent connections per host. Limits apply to active connections because it's not possible to detect a successful TCP handshake in Javascript.

dustinblackman commented 6 years ago

Will take a look at this later in the week. Thanks!

dustinblackman commented 6 years ago

This is brilliant and works exactly as expected. Thanks a lot!