Apply CS fixes.

src/FreeDSx/Ldap/Control/Control.php

@@ -81,7 +81,7 @@ class Control implements ProtocolElementInterface, Stringable
     public function __construct(
         protected string $controlType,
         protected bool $criticality = false,
-        protected AbstractType|ProtocolElementInterface|string|null $controlValue = null
+        protected AbstractType | ProtocolElementInterface | string | null $controlValue = null
     ) {
     }
 

src/FreeDSx/Ldap/LdapClient.php

@@ -38,6 +38,7 @@
 use FreeDSx\Ldap\Search\Vlv;
 use FreeDSx\Ldap\Sync\SyncRepl;
 use FreeDSx\Sasl\Exception\SaslException;
+use SensitiveParameter;
 use Stringable;
 
 /**
@@ -74,7 +75,7 @@ public function __construct(
      */
     public function bind(
         string $username,
-        #[\SensitiveParameter]
+        #[SensitiveParameter]
         string $password,
     ): LdapMessageResponse {
         return $this->sendAndReceive(
@@ -93,7 +94,7 @@ public function bind(
      * @throws SaslException
      */
     public function bindSasl(
-        #[\SensitiveParameter]
+        #[SensitiveParameter]
         array $options = [],
         string $mechanism = ''
     ): LdapMessageResponse {

src/FreeDSx/Ldap/Operation/Request/SearchRequest.php

@@ -338,7 +338,7 @@ public function useReferralHandler(?Closure $referralHandler): self
      */
     public function useCancelStrategy(string $strategy): self
     {
-        if ($strategy !== self::CANCEL_STOP && $strategy!== self::CANCEL_CONTINUE) {
+        if ($strategy !== self::CANCEL_STOP && $strategy !== self::CANCEL_CONTINUE) {
             throw new UnexpectedValueException(sprintf(
                 'The cancel strategy must be one of: %s',
                 implode(
@@ -387,7 +387,8 @@ public static function fromAsn1(AbstractType $type): self
         }
         $filter = FilterFactory::get($filter);
 
-        if (!($baseDn instanceof OctetStringType
+        if (
+            !($baseDn instanceof OctetStringType
             && $scope instanceof EnumeratedType
             && $deref instanceof EnumeratedType
             && $sizeLimit instanceof IntegerType

src/FreeDSx/Ldap/Operation/Request/SimpleBindRequest.php

@@ -16,6 +16,7 @@
 use FreeDSx\Asn1\Asn1;
 use FreeDSx\Asn1\Type\AbstractType;
 use FreeDSx\Ldap\Exception\BindException;
+use SensitiveParameter;
 
 /**
  * Represents a simple bind request consisting of a username (dn, etc) and a password.
@@ -34,7 +35,7 @@ class SimpleBindRequest extends BindRequest
 
     public function __construct(
         string $username,
-        #[\SensitiveParameter]
+        #[SensitiveParameter]
         string $password,
         int $version = 3
     ) {

src/FreeDSx/Ldap/Operations.php

@@ -38,6 +38,7 @@
 use FreeDSx\Ldap\Protocol\ProtocolElementInterface;
 use FreeDSx\Ldap\Search\Filter\FilterInterface;
 use FreeDSx\Ldap\Search\Filters;
+use SensitiveParameter;
 use Stringable;
 
 /**
@@ -68,7 +69,7 @@ public static function add(Entry $entry): AddRequest
      */
     public static function bind(
         string $username,
-        #[\SensitiveParameter]
+        #[SensitiveParameter]
         string $password,
     ): SimpleBindRequest {
         return new SimpleBindRequest(

src/FreeDSx/Ldap/Protocol/ClientProtocolHandler/ClientReferralHandler.php

@@ -29,7 +29,6 @@
 use FreeDSx\Ldap\Operation\ResultCode;
 use FreeDSx\Ldap\Protocol\LdapMessageRequest;
 use FreeDSx\Ldap\Protocol\LdapMessageResponse;
-use FreeDSx\Ldap\Protocol\Queue\ClientQueue;
 use FreeDSx\Ldap\Protocol\ReferralContext;
 use FreeDSx\Ldap\ReferralChaserInterface;
 use FreeDSx\Ldap\Search\Filters;

src/FreeDSx/Ldap/Protocol/ClientProtocolHandler/RequestCanceler.php

@@ -36,7 +36,7 @@ public function __construct(
         private readonly string $strategy = SearchRequest::CANCEL_STOP,
         ?Closure $messageProcessor = null,
     ) {
-        $this->messageProcessor = $messageProcessor ?? fn() => null;
+        $this->messageProcessor = $messageProcessor ?? fn () => null;
     }
 
     /**

src/FreeDSx/Ldap/Protocol/Factory/ClientProtocolHandlerFactory.php

@@ -92,7 +92,7 @@ public function forResponse(
             return new ClientProtocolHandler\ClientBasicHandler($this->queue());
         }
     }
-    
+
     private function queue(): ClientQueue
     {
         return $this->queueInstantiator->make();

src/FreeDSx/Ldap/Protocol/ServerProtocolHandler/ServerBindHandler.php

@@ -18,7 +18,6 @@
 use FreeDSx\Ldap\Operation\Request\BindRequest;
 use FreeDSx\Ldap\Operation\Request\SimpleBindRequest;
 use FreeDSx\Ldap\Operation\ResultCode;
-use FreeDSx\Ldap\Protocol\Factory\ResponseFactory;
 use FreeDSx\Ldap\Protocol\LdapMessageRequest;
 use FreeDSx\Ldap\Protocol\Queue\ServerQueue;
 use FreeDSx\Ldap\Server\RequestHandler\RequestHandlerInterface;

src/FreeDSx/Ldap/Server/RequestHandler/GenericRequestHandler.php

@@ -23,6 +23,7 @@
 use FreeDSx\Ldap\Operation\Request\ModifyRequest;
 use FreeDSx\Ldap\Operation\Request\SearchRequest;
 use FreeDSx\Ldap\Server\RequestContext;
+use SensitiveParameter;
 
 /**
  * This allows the LDAP server to run, but rejects everything. You can extend and selectively override specific request
@@ -44,7 +45,7 @@ public function add(
 
     public function bind(
         string $username,
-        #[\SensitiveParameter]
+        #[SensitiveParameter]
         string $password,
     ): bool {
         return false;

src/FreeDSx/Ldap/Server/RequestHandler/ProxyRequestHandler.php

@@ -28,6 +28,7 @@
 use FreeDSx\Ldap\Operation\Request\SearchRequest;
 use FreeDSx\Ldap\Operation\ResultCode;
 use FreeDSx\Ldap\Server\RequestContext;
+use SensitiveParameter;
 
 /**
  * Proxies requests to an LDAP server and returns the response. You should extend this to add your own constructor and
@@ -51,7 +52,7 @@ public function __construct(ClientOptions $options = new ClientOptions())
      */
     public function bind(
         string $username,
-        #[\SensitiveParameter]
+        #[SensitiveParameter]
         string $password,
     ): bool {
         try {

src/FreeDSx/Ldap/Server/RequestHandler/RequestHandlerInterface.php

@@ -23,6 +23,7 @@
 use FreeDSx\Ldap\Operation\Request\ModifyRequest;
 use FreeDSx\Ldap\Operation\Request\SearchRequest;
 use FreeDSx\Ldap\Server\RequestContext;
+use SensitiveParameter;
 
 /**
  * Handler methods for LDAP server requests.
@@ -110,7 +111,7 @@ public function search(
      */
     public function bind(
         string $username,
-        #[\SensitiveParameter]
+        #[SensitiveParameter]
         string $password,
     ): bool;
 }

src/FreeDSx/Ldap/Server/Token/BindToken.php

@@ -13,6 +13,8 @@
 
 namespace FreeDSx\Ldap\Server\Token;
 
+use SensitiveParameter;
+
 /**
  * Represents a username/password token that is bound and authorized.
  *
@@ -28,7 +30,7 @@ class BindToken implements TokenInterface
 
     public function __construct(
         string $username,
-        #[\SensitiveParameter]
+        #[SensitiveParameter]
         string $password,
         int $version = 3
     ) {

tests/integration/FreeDSx/Ldap/Search/PagingTest.php

@@ -88,7 +88,7 @@ public function testPagingAllWhenEntryHandlerIsUsed(): void
             ),
             'cn'
         );
-        $operation->useEntryHandler(fn(EntryResult $result)=> $entries->add($result->getEntry()));
+        $operation->useEntryHandler(fn (EntryResult $result) => $entries->add($result->getEntry()));
 
         $this->paging = $this->client->paging($operation);
 

tests/spec/FreeDSx/Ldap/ContainerSpec.php

@@ -18,7 +18,6 @@
 use FreeDSx\Ldap\LdapClient;
 use FreeDSx\Ldap\Protocol\ClientProtocolHandler;
 use FreeDSx\Ldap\Protocol\Factory\ClientProtocolHandlerFactory;
-use FreeDSx\Ldap\Protocol\Factory\ServerProtocolHandlerFactory;
 use FreeDSx\Ldap\Protocol\Queue\ClientQueueInstantiator;
 use FreeDSx\Ldap\Protocol\RootDseLoader;
 use FreeDSx\Ldap\Protocol\ServerAuthorization;

tests/spec/FreeDSx/Ldap/LdapClientSpec.php

@@ -18,7 +18,6 @@
 use FreeDSx\Ldap\Entry\Entries;
 use FreeDSx\Ldap\Entry\Entry;
 use FreeDSx\Ldap\Exception\OperationException;
-use FreeDSx\Ldap\LdapClient;
 use FreeDSx\Ldap\Operation\LdapResult;
 use FreeDSx\Ldap\Operation\Request\ExtendedRequest;
 use FreeDSx\Ldap\Operation\Request\SimpleBindRequest;

tests/spec/FreeDSx/Ldap/Protocol/ClientProtocolHandler/ClientBasicHandlerSpec.php

@@ -13,7 +13,6 @@
 
 namespace spec\FreeDSx\Ldap\Protocol\ClientProtocolHandler;
 
-use FreeDSx\Ldap\ClientOptions;
 use FreeDSx\Ldap\Exception\BindException;
 use FreeDSx\Ldap\Operation\LdapResult;
 use FreeDSx\Ldap\Operation\Request\CompareRequest;
@@ -55,7 +54,7 @@ public function it_should_implement_RequestHandlerInterface(): void
         $this->shouldBeAnInstanceOf(RequestHandlerInterface::class);
     }
 
-    public function it_should_handle_a_request_and_return_a_response(ClientQueue $queue,): void
+    public function it_should_handle_a_request_and_return_a_response(ClientQueue $queue, ): void
     {
         $queue->sendMessage(Argument::type(LdapMessageRequest::class))->shouldBeCalledOnce();
         $queue->getMessage(1)->willReturn(

tests/spec/FreeDSx/Ldap/Protocol/ClientProtocolHandler/ClientSaslBindHandlerSpec.php

@@ -54,7 +54,7 @@ public function let(
             ->willReturn(
                 Entry::fromArray(
                     '',
-                    ['supportedSaslMechanisms' => ['DIGEST-MD5', 'CRAM-MD5'],]
+                    ['supportedSaslMechanisms' => ['DIGEST-MD5', 'CRAM-MD5'], ]
                 )
             );
         $queue

tests/spec/FreeDSx/Ldap/Protocol/ClientProtocolHandler/MockCancelResponseProcessor.php

@@ -22,7 +22,7 @@
  */
 class MockCancelResponseProcessor
 {
-    public function __invoke(LdapMessageResponse $messageResponse,): void
+    public function __invoke(LdapMessageResponse $messageResponse): void
     {
     }
 }

tests/spec/FreeDSx/Ldap/Protocol/ClientProtocolHandler/RequestCancelerSpec.php

@@ -20,17 +20,14 @@
 use FreeDSx\Ldap\Operation\Request\CancelRequest;
 use FreeDSx\Ldap\Operation\Request\SearchRequest;
 use FreeDSx\Ldap\Operation\Response\ExtendedResponse;
-use FreeDSx\Ldap\Operation\Response\SearchResultDone;
 use FreeDSx\Ldap\Operation\Response\SearchResultEntry;
 use FreeDSx\Ldap\Operation\Response\SearchResultReference;
 use FreeDSx\Ldap\Operation\ResultCode;
 use FreeDSx\Ldap\Protocol\LdapMessageRequest;
 use FreeDSx\Ldap\Protocol\LdapMessageResponse;
 use FreeDSx\Ldap\Protocol\Queue\ClientQueue;
-use PhpParser\Node\Arg;
 use PhpSpec\ObjectBehavior;
 use Prophecy\Argument;
-use spec\FreeDSx\Ldap\TestFactoryTrait;
 
 class RequestCancelerSpec extends ObjectBehavior
 {
@@ -54,7 +51,7 @@ public function it_should_return_the_cancel_response(ClientQueue $queue): void
 
         $queue
             ->sendMessage(Argument::that(
-                fn(LdapMessageRequest $request) =>
+                fn (LdapMessageRequest $request) =>
                     $request->getRequest() instanceof CancelRequest
             ))
             ->shouldBeCalledOnce();

tests/spec/FreeDSx/Ldap/Protocol/Factory/ServerProtocolHandlerFactorySpec.php

@@ -18,7 +18,6 @@
 use FreeDSx\Ldap\Entry\Entry;
 use FreeDSx\Ldap\Operation\Request\ExtendedRequest;
 use FreeDSx\Ldap\Operations;
-use FreeDSx\Ldap\Protocol\ClientProtocolHandler\RequestHandlerInterface;
 use FreeDSx\Ldap\Protocol\Factory\ServerProtocolHandlerFactory;
 use FreeDSx\Ldap\Protocol\Queue\ServerQueue;
 use FreeDSx\Ldap\Protocol\ServerProtocolHandler\ServerDispatchHandler;
@@ -116,7 +115,7 @@ public function it_should_get_an_unbind_handler(ServerQueue $queue): void
             ->shouldBeAnInstanceOf(ServerUnbindHandler::class);
     }
 
-    public function it_should_get_the_dispatch_handler_for_common_requests(HandlerFactoryInterface $handlerFactory,): void
+    public function it_should_get_the_dispatch_handler_for_common_requests(HandlerFactoryInterface $handlerFactory, ): void
     {
         $handlerFactory->makeRequestHandler()
             ->willReturn(new GenericRequestHandler());

tests/spec/FreeDSx/Ldap/Protocol/ServerAuthorizationSpec.php

@@ -90,7 +90,7 @@ public function it_should_require_authentication_for_all_other_operations(): voi
     public function it_should_not_require_authentication_if_it_has_been_explicitly_disabled(): void
     {
         $this->beConstructedWith(
-            (new ServerOptions)
+            (new ServerOptions())
                 ->setAllowAnonymous(false)
                 ->setRequireAuthentication(false),
             new AnonToken()
@@ -116,7 +116,7 @@ public function it_should_not_allow_anonymous_authentication_by_default(): void
     public function it_should_respect_the_option_for_whether_anon_binds_are_allowed(): void
     {
         $this->beConstructedWith(
-            (new ServerOptions)
+            (new ServerOptions())
                 ->setAllowAnonymous(true),
             new AnonToken()
         );

tests/spec/FreeDSx/Ldap/Protocol/ServerProtocolHandler/ServerStartTlsHandlerSpec.php

@@ -21,7 +21,6 @@
 use FreeDSx\Ldap\Protocol\LdapMessageResponse;
 use FreeDSx\Ldap\Protocol\Queue\ServerQueue;
 use FreeDSx\Ldap\Protocol\ServerProtocolHandler\ServerStartTlsHandler;
-use FreeDSx\Ldap\Server\RequestHandler\RequestHandlerInterface;
 use FreeDSx\Ldap\Server\Token\TokenInterface;
 use FreeDSx\Ldap\ServerOptions;
 use PhpSpec\ObjectBehavior;

tests/spec/FreeDSx/Ldap/Protocol/ServerProtocolHandler/ServerWhoAmIHandlerSpec.php

@@ -20,7 +20,6 @@
 use FreeDSx\Ldap\Protocol\LdapMessageResponse;
 use FreeDSx\Ldap\Protocol\Queue\ServerQueue;
 use FreeDSx\Ldap\Protocol\ServerProtocolHandler\ServerWhoAmIHandler;
-use FreeDSx\Ldap\Server\RequestHandler\RequestHandlerInterface;
 use FreeDSx\Ldap\Server\Token\AnonToken;
 use FreeDSx\Ldap\Server\Token\BindToken;
 use PhpSpec\ObjectBehavior;