project-slippi / Ishiiruka

GNU General Public License v2.0
330 stars 142 forks source link

Bump Game Reporter connection pool to 5 to match old C++ CURL config #396

Closed ryanmcgrath closed 1 year ago

ryanmcgrath commented 1 year ago

I suspect this was an oversight on my part when doing the port, but I think we could - in somewhat rare circumstances - encounter a crash if we're trying to execute calls on separate threads at once. CURL appears to have 5 as the default for the pool, which I missed - so I think it's probably ideal to just do that here for sanity's sake.