Page MenuHomePhabricator

[SPIKE] Review and enable rate limit support for AQS 1.0 in service layer
Closed, ResolvedPublic

Description

Since AQS 1.0 is a RB-ish service, review see if it's possible to enable rate limit support at the service layer otherwise we'll default to relying on REST-Gateway for this.

Event Timeline

DAlangi_WMF changed the task status from Open to Stalled.Jul 11 2023, 10:56 AM

After some fiddling, this is proving stubborn. I'll create a task to do this at the gateway level because we really do want all services to be rate limited. So this will be common for all services.

RB rate limiter seems to be unreliable and I've not been able to make it work for AQS 1.0. I'm now relying on support for this via the REST-Gateway per endpoint.

MSantos subscribed.

I'm removing RESTBase Sunsetting as this is not needed for its completion. Please, re-add it in case I'm missing something.

Also, I'm not seeing a tag for AQS1.0 so I'm adding AQS2.0 for review.

hnowlan claimed this task.

I am going to close this issue as AQS1 is no longer in service.