Benzine/Docker-Swarm-Loadbalancer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix php-stan

Browse source
This commit is contained in:
Greyscale 2024-05-19 02:47:17 +02:00
parent e1d8a91c07
commit 30a3a3bcb8
No known key found for this signature in database
GPG key ID: 74BAFF55434DA4B2
1 changed files with 1 additions and 2 deletions

3
src/Bouncer.php
View file

@ -338,7 +338,7 @@ class Bouncer
// if this bouncerTarget already exists, merge it in instead of adding it.
foreach ($bouncerTargets as $existing) {
if ($existing->getDomains() == $bouncerTarget->getDomains()) {
if (isset($bouncerTarget) && $existing->getDomains() == $bouncerTarget->getDomains()) {
$this->logger->debug('Found another instance of the same service, merging them together.', ['emoji' => Emoji::cupcake()]);
$existing->setEndpoints(array_merge($existing->getEndpoints(), $bouncerTarget->getEndpoints()));
unset($bouncerTarget);
@ -351,7 +351,6 @@ class Bouncer
// Iterate over bouncers and check validity
$validBouncerTargets = [];
foreach ($bouncerTargets as $bouncerTarget) {
// @phpstan-ignore-next-line MB: I'm not sure you're right about ->hasCustomNginxConfig only returning false, Stan..
if ($bouncerTarget->isEndpointValid() || $bouncerTarget->hasCustomNginxConfig()) {
$validBouncerTargets[] = $bouncerTarget;
} else {