symfony
diff --git a/‎src/Symfony/Contracts/Service/ServiceSubscriberTrait.php
Copy file name to clipboardExpand all lines: src/Symfony/Contracts/Service/ServiceSubscriberTrait.php
+1-1Lines changed: 1 addition & 1 deletion b/‎src/Symfony/Contracts/Service/ServiceSubscriberTrait.php
Copy file name to clipboardExpand all lines: src/Symfony/Contracts/Service/ServiceSubscriberTrait.php
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Symfony/Contracts/Tests/Service/ServiceSubscriberTraitTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Contracts/Tests/Service/ServiceSubscriberTraitTest.php
+13-2Lines changed: 13 additions & 2 deletions b/‎src/Symfony/Contracts/Tests/Service/ServiceSubscriberTraitTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Contracts/Tests/Service/ServiceSubscriberTraitTest.php
+13-2Lines changed: 13 additions & 2 deletions
@@ -51,7 +51,7 @@ public static function getSubscribedServices(): array
             $attribute = $attribute->newInstance();
             $attribute->key ??= self::class.'::'.$method->name;
             $attribute->type ??= $returnType instanceof \ReflectionNamedType ? $returnType->getName() : (string) $returnType;
-            $attribute->nullable = $returnType->allowsNull();
+            $attribute->nullable = $attribute->nullable ?: $returnType->allowsNull();
 
             if ($attribute->attributes) {
                 $services[] = $attribute;
 
@@ -27,7 +27,8 @@ public function testMethodsOnParentsAndChildrenAreIgnoredInGetSubscribedServices
     {
         $expected = [
             TestService::class.'::aService' => Service2::class,
-            TestService::class.'::nullableService' => '?'.Service2::class,
+            TestService::class.'::nullableInAttribute' => '?'.Service2::class,
+            TestService::class.'::nullableReturnType' => '?'.Service2::class,
             new SubscribedService(TestService::class.'::withAttribute', Service2::class, true, new Required()),
         ];
 
@@ -103,8 +104,18 @@ public function aService(): Service2
     {
     }
 
+    #[SubscribedService(nullable: true)]
+    public function nullableInAttribute(): Service2
+    {
+        if (!$this->container->has(__METHOD__)) {
+            throw new \LogicException();
+        }
+
+        return $this->container->get(__METHOD__);
+    }
+
     #[SubscribedService]
-    public function nullableService(): ?Service2
+    public function nullableReturnType(): ?Service2
     {
     }
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,8 @@ public function testMethodsOnParentsAndChildrenAreIgnoredInGetSubscribedServices`
`27`	`27`	`{`
`28`	`28`	`$expected = [`
`29`	`29`	`TestService::class.'::aService' => Service2::class,`
`30`		`- TestService::class.'::nullableService' => '?'.Service2::class,`
	`30`	`+ TestService::class.'::nullableInAttribute' => '?'.Service2::class,`
	`31`	`+ TestService::class.'::nullableReturnType' => '?'.Service2::class,`
`31`	`32`	`new SubscribedService(TestService::class.'::withAttribute', Service2::class, true, new Required()),`
`32`	`33`	`];`
`33`	`34`
`@@ -103,8 +104,18 @@ public function aService(): Service2`
`103`	`104`	`{`
`104`	`105`	`}`
`105`	`106`
	`107`	`+ #[SubscribedService(nullable: true)]`
	`108`	`+ public function nullableInAttribute(): Service2`
	`109`	`+ {`
	`110`	`+ if (!$this->container->has(__METHOD__)) {`
	`111`	`+ throw new \LogicException();`
	`112`	`+ }`
	`113`	`+`
	`114`	`+ return $this->container->get(__METHOD__);`
	`115`	`+ }`
	`116`	`+`
`106`	`117`	`#[SubscribedService]`
`107`		`- public function nullableService(): ?Service2`
	`118`	`+ public function nullableReturnType(): ?Service2`
`108`	`119`	`{`
`109`	`120`	`}`
`110`	`121`