[12.x] Refactor the setTrustedProxyIpAddresses() method

[AhmedAlaa4611]

Description

This PR avoid redundant calls and simplify checks by:

• Storing $request->host() in a local $host variable instead of calling it multiple times.
• Replacing repeated str_ends_with($host, '.on-forge.com') || str_ends_with($host, '.on-vapor.com') checks with a single $isForgeOrVapor variable.

This reduces duplication and makes the intent of the code clearer without changing its behavior.

[tontonsb]

So what's the performance difference? Is it in memory usage or in speed? I mean you've introduced a new variable that will have to be refcounted, garbagecollected and so on.

[AhmedAlaa4611]

So what's the performance difference? Is it in memory usage or in speed? I mean you've introduced a new variable that will have to be refcounted, garbagecollected and so on.

Instead of calling the host() method multiple times just call it once.

[AhmedAlaa4611]

I will draft this until I implement the @shaedrich suggestions.

[AhmedAlaa4611]

It's now ready for review. Also, I have changed the description to match the new changes.

[GrahamCampbell]

This makes zero performance difference in the real world.

[AhmedAlaa4611]

This makes zero performance difference in the real world.

I have changed the PR title to refactor.

[taylorotwell]

Thanks for your pull request to Laravel!

Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.

If applicable, please consider releasing your code as a package so that the community can still take advantage of your contributions!