VNG-Realisatie / Haal-Centraal-BRK-bevragen

Project repository tbv de ontwikkeling van de Haal Centraal BRK Bevragen API
https://vng-realisatie.github.io/Haal-Centraal-BRK-bevragen/
Other
19 stars 15 forks source link

Verhoging rate limit voor applicatie met meerdere gebruikers #994

Open teije-processfive opened 7 months ago

teije-processfive commented 7 months ago

Beschrijf de verbetering We lopen tegen een probleem aan met betrekking tot de rate limitng op de BRK API in ons platform. Onze applicatie maakt gebruik van één API key, maar omdat meerdere gebruikers tegelijkertijd het platform kunnen gebruiken, zijn deze collectief beperkt tot 5 aanvragen per seconde.

Dit kan ervoor zorgen dat zodra meerdere gebruikers individuele aanvragen doen, onze applicatie niet naar behoren kan functioneren. Daarbij bestaan er features binnen de applicatie waarvoor er meerdere aanvragen in een keer verstuurd worden (mogelijk meer dan 5), hiermee kan een enkele gebruiker het limiet al overschrijden.

We vragen daarom om een verhoging van ons rate limit naar een niveau dat onze applicatie effectief ondersteunt. Hiermee kunnen we een soepele werking garanderen, zelfs bij gelijktijdig gebruik door meerdere gebruikers.

Met vriendelijke groet,

Teije van Hemert Processfive B.V. I: www.processfive.com

teije-processfive commented 7 months ago

@HildeVos: Ik zie dat jij toewezen bent aan dit item. Aangezien wij op het moment tegen de ratelimit aanlopen op productie ben ik benieuwd wanneer je tijd hebt om hier naar te kijken.

HildeVos commented 7 months ago

Ik snap dat jullie hier nu tegenaan lopen en ga kijken wat mogelijk is. Ik moet dit echter eerst afstemmen met oa. onze productmanager (die momenteel op vakantie is, volgende week is ze er weer). We hebben de rate limit nu namelijk al hoger staan dan hetgeen is afgesproken in de SLA, dus ik kan het niet zomaar aanpassen. Wordt vervolgd!

teije-processfive commented 7 months ago

Bedankt voor je reactie en de moeite, ik hoor graag wat er mogelijk is zodra je het weet!

teije-processfive commented 7 months ago

Nog een extra vraag; je geeft aan dat de ratelimit al hoger staat, kun je aangeven welk limiet er nu ingesteld is?

HildeVos commented 7 months ago

Het staat nu ingesteld op 5 per seconde en 30.000 per dag (per organisatie). In de SLA is max 35 per minuut en max 5000 per dag afgesproken.

teije-processfive commented 7 months ago

Oke, duidelijk. Ik wacht verdere updates hierover dan even af!

teije-processfive commented 7 months ago

@HildeVos

Als het goed is is bij jullie de productowner weer aan de slag, daarom hieronder nog even puntsgewijs onze casus:

Als er meer info nodig is hoor ik het natuurlijk graag!

HildeVos commented 7 months ago

Ik heb het net besproken met onze productmanager. Is het voor jullie werkbaar als we de rate limit (per seconde) verdubbelen? Dus dat het dan 10 per seconde worden. BRK Bevragen is alleen bedoeld voor individuele bevragingen, dus in theorie zou een organisatie met die aantallen moeten kunnen werken.

De rate limit is gekoppeld aan een API key. Als het goed is heeft elke organisatie een eigen API key, dus zou de rate limit per organisatie moeten zijn. En dus niet per gebruiker van jullie koppeling.

teije-processfive commented 7 months ago

Verdubbelen zou voor nu het directe probleem even oplossen, dus als dat ingesteld kan worden, heel graag!

Op het moment hebben we dus inderdaad 32 organisaties die gedeeld gebruik maken van 1 API key. Ik begrijp dat eigenlijk elke organisatie zijn eigen key zou moeten hebben en vanzelfsprekend als deze organisaties ook daadwerkelijk individueel bevragingen doen.

Het voelt echter als een beheerlast die zijn doel voorbij schiet wanneer we in onze applicatie 32 keer een API moeten configureren die elk in exact dezelfde omgeving leeft als onze huidige API key.

HildeVos commented 6 months ago

De wijziging is doorgevoerd. De limiet is nu bijgewerkt naar 10 per seconde.