Fix recursion in Container Proxy (#1140)

Author: spiralbot

src/Exception/Container/RecursiveProxyException.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Spiral\Core\Exception\Container;
+
+/**
+ * Recursion can occur due to improper container configuration or
+ * an unplanned exit from the scope by the execution thread.
+ */
+class RecursiveProxyException extends ContainerException
+{
+}

src/Internal/Proxy.php

@@ -86,4 +86,9 @@ public static function create(
 
         return $instance;
     }
+
+    public static function isProxy(object $object): bool
+    {
+        return \in_array($object::class, self::$classes, true);
+    }
 }

src/Internal/Proxy/Resolver.php

@@ -7,6 +7,8 @@
 use Psr\Container\ContainerInterface;
 use Spiral\Core\ContainerScope;
 use Spiral\Core\Exception\Container\ContainerException;
+use Spiral\Core\Exception\Container\RecursiveProxyException;
+use Spiral\Core\Internal\Proxy;
 
 /**
  * @internal
@@ -32,6 +34,12 @@ public static function resolve(
             );
         }
 
+        if (Proxy::isProxy($result)) {
+            throw new RecursiveProxyException(
+                \sprintf('Recursive proxy detected for `%s`.', $alias),
+            );
+        }
+
         return $result;
     }
 }

tests/Scope/ProxyTest.php

@@ -9,6 +9,7 @@
 use Spiral\Core\Attribute\Proxy;
 use Spiral\Core\Container;
 use Spiral\Core\Container\InjectorInterface;
+use Spiral\Core\Exception\Container\RecursiveProxyException;
 use Spiral\Core\Scope;
 use Spiral\Tests\Core\Scope\Stub\Context;
 use Spiral\Tests\Core\Scope\Stub\ContextInterface;
@@ -296,6 +297,22 @@ public function __toString(): string
         );
     }
 
+    /**
+     * Proxy gets a proxy of the same type.
+     */
+    public function testRecursiveProxy(): void
+    {
+        $root = new Container();
+        $root->bind(UserInterface::class, new \Spiral\Core\Config\Proxy(UserInterface::class));
+
+        $this->expectException(RecursiveProxyException::class);
+
+        $root->runScope(
+            new Scope(),
+            fn(#[Proxy] UserInterface $user) => $user->getName(),
+        );
+    }
+
     /*
     // Proxy::$attachContainer=true tests