Handle 429 error message : TOO MANY REQUESTS on website.domain

@lirone the throttling is done by a middleware group, you can create a new group for the routes that you don't want to throttle

also look at

https://laravel.com/docs/8.x/releases#improved-rate-limiting

Thank for the link.

I tried sommething like this

RateLimiter::for('api', function (Request $request) {
            if ($request->ip() == "IP") {
                return Limit::perMinute(1000)->by(optional($request->user())->id ?: $request->ip());
            }
            return Limit::perMinute(60)->by(optional($request->user())->id ?: $request->ip());
        });

But it does not work. Did I miss something ?

@lirone

does not work.

is it not matching your ip?

if put a dd($request->ip()) just inside that method what do you see?

Ok it works ! I made a mistake on the IP adress as I was working on local. My bad.

I have a last question. I had difficulties to understand this line

by(optional($request->user())->id ?: $request->ip())

Can you please explicit me more ?

@lirone by() is a method on the Limit class that filters by user.

The

(optional($request->user())->id

is checking if you have a user, and if so, getting the id of it as if you just did

$request->user()->id

and you didn't have a user, then you would getting an error as you would be trying to get an id off an object that didn't exist.

its the same as doing:

$id = isset($request->user()) ? $request->user()->id : null;

or old school

$id = null;

if(isset($request->user()){
$id = $request->user()->id; 
}

And so if the users does not exists, it take the IP right ?

@lirone correct.