Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verhoging rate limit voor applicatie met meerdere gebruikers #994

Open
teije-processfive opened this issue Jan 17, 2024 · 10 comments
Open
Assignees

Comments

@teije-processfive
Copy link

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
Copy link
Author

@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
Copy link
Collaborator

HildeVos commented Feb 1, 2024

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
Copy link
Author

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

@teije-processfive
Copy link
Author

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

@HildeVos
Copy link
Collaborator

HildeVos commented Feb 1, 2024

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
Copy link
Author

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

@teije-processfive
Copy link
Author

teije-processfive commented Feb 6, 2024

@HildeVos

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

  • De huidige is ratelimit gebaseerd op een enkele organisatie, maar er maken 32 organisaties (en daarbinenn meerdere gebruikers) tegelijkertijd gebruik van onze koppeling. Met dit in gedachten zou een ratelimit van 32 x 5 seconden passender zijn voor ons gebruik.

  • In sommige gevallen lopen we nu al tegen een ratelimit probleem aan met een enkele gebruiker (ophalen van meer dan 5 kadastrale gegevens bij een object)

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

@HildeVos
Copy link
Collaborator

HildeVos commented Feb 6, 2024

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
Copy link
Author

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
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants