tileRequestLimit: limit concurrent connections

[StephanGeorg]

Due to new limitations of the OSM tiles usage policy I added tilesRequestLimit in staticmaps@1.3.0 option to limit concurrent connections to the tiles server. This value is set to 2 by default and can be overwritten or disabled with tilesRequestLimit: 0.

[StephanGeorg]

@OlivierKamers FYI

[OlivierKamers]

Nice 👍
I also saw the update to get rid of the request library. I played around with it a few weeks ago because we had issues parallelising multiple map renderings. I added an option to provide a custom tile loader function (so we can use our blob storage functions and some memoization) but it looks like the got library you're using now might already take care of the issues we had. I haven't been able to clean up the implementation yet so that's why I didn't propose the change here, and it's just on a branch in our fork: Sitemark@5084341

[StephanGeorg]

What kind of issues?

[OlivierKamers]

I had to limit the number of requests to avoid timeouts from the request library. Externalising the function also allowed me to use a cache for tiles that are requested more than once.

[StephanGeorg]

Ok, that could indeed be an improvement. I thought for now caching should be done at the tiles server side, for example via CDN.