[EasyApiToken] Remove deprecated code (#449)

packages/EasyApiToken/src/Bridge/Laravel/EasyApiTokenServiceProvider.php

@@ -7,9 +7,7 @@
 use EonX\EasyApiToken\Bridge\BridgeConstantsInterface;
 use EonX\EasyApiToken\Factories\ApiTokenDecoderFactory;
 use EonX\EasyApiToken\Interfaces\ApiTokenDecoderInterface;
-use EonX\EasyApiToken\Interfaces\ApiTokenDecoderProviderInterface;
 use EonX\EasyApiToken\Interfaces\Factories\ApiTokenDecoderFactoryInterface as DecoderFactoryInterface;
-use EonX\EasyApiToken\Providers\FromConfigDecoderProvider;
 use Illuminate\Support\ServiceProvider;
 
 final class EasyApiTokenServiceProvider extends ServiceProvider
@@ -25,24 +23,6 @@ public function register(): void
     {
         $this->mergeConfigFrom(__DIR__ . '/config/easy-api-token.php', 'easy-api-token');
 
-        // TODO - Remove in 3.0.
-        if (empty(\config('easy-api-token.decoders', [])) === false) {
-            @\trigger_error(\sprintf(
-                'Defining ApiTokenDecoders using a config file is deprecated since 2.4 and will be removed in 3.0.
-                Use %s instead.',
-                ApiTokenDecoderProviderInterface::class
-            ), \E_USER_NOTICE);
-
-            $this->app->singleton(FromConfigDecoderProvider::class, function (): FromConfigDecoderProvider {
-                return new FromConfigDecoderProvider(
-                    \config('easy-api-token.decoders', []),
-                    \config('easy-api-token.factories'),
-                    \config('easy-api-token.default_decoder')
-                );
-            });
-            $this->app->tag(FromConfigDecoderProvider::class, [BridgeConstantsInterface::TAG_DECODER_PROVIDER]);
-        }
-
         $this->app->singleton(ApiTokenDecoderInterface::class, function (): ApiTokenDecoderInterface {
             return $this->app->make(DecoderFactoryInterface::class)->buildDefault();
         });

packages/EasyApiToken/src/Bridge/Symfony/DependencyInjection/EasyApiTokenExtension.php

@@ -23,40 +23,8 @@ public function load(array $configs, ContainerBuilder $container): void
         $loader = new PhpFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
         $loader->load('services.php');
 
-        $container->registerForAutoconfiguration(ApiTokenDecoderProviderInterface::class)
+        $container
+            ->registerForAutoconfiguration(ApiTokenDecoderProviderInterface::class)
             ->addTag(BridgeConstantsInterface::TAG_DECODER_PROVIDER);
-
-        // Resolve config
-        $decoders = [];
-        $defaultFactories = null;
-        $defaultDecoder = null;
-
-        foreach ($configs as $config) {
-            if (isset($config['decoders'])) {
-                $decoders = $config['decoders'];
-            }
-
-            if (isset($config['default_decoder'])) {
-                $defaultDecoder = $config['default_decoder'];
-            }
-
-            if (isset($config['default_factories'])) {
-                $defaultFactories = $config['default_factories'];
-            }
-        }
-
-        if (empty($decoders) === false) {
-            @\trigger_error(\sprintf(
-                'Defining ApiTokenDecoders using a config file is deprecated since 2.4 and will be removed in 3.0.
-                Use %s instead.',
-                ApiTokenDecoderProviderInterface::class
-            ), \E_USER_NOTICE);
-
-            $container->setParameter(BridgeConstantsInterface::PARAM_DECODERS, $decoders);
-            $container->setParameter(BridgeConstantsInterface::PARAM_DEFAULT_FACTORIES, $defaultFactories);
-            $container->setParameter(BridgeConstantsInterface::PARAM_DEFAULT_DECODER, $defaultDecoder);
-
-            $loader->load('from_config_provider.php');
-        }
     }
 }

packages/EasyApiToken/src/Bridge/Symfony/Resources/config/services.php

@@ -8,7 +8,6 @@
 use EonX\EasyApiToken\Factories\ApiTokenDecoderFactory;
 use EonX\EasyApiToken\Interfaces\ApiTokenDecoderInterface;
 use EonX\EasyApiToken\Interfaces\Factories\ApiTokenDecoderFactoryInterface;
-use EonX\EasyApiToken\Interfaces\Factories\EasyApiTokenDecoderFactoryInterface;
 
 return static function (ContainerConfigurator $container): void {
     $services = $container->services();
@@ -20,8 +19,6 @@
         ->set(ApiTokenDecoderFactoryInterface::class, ApiTokenDecoderFactory::class)
         ->arg('$decoderProviders', tagged_iterator(BridgeConstantsInterface::TAG_DECODER_PROVIDER));
 
-    $services->alias(EasyApiTokenDecoderFactoryInterface::class, ApiTokenDecoderFactoryInterface::class);
-
     $services
         ->set(ApiTokenDecoderInterface::class)
         ->factory([ref(ApiTokenDecoderFactoryInterface::class), 'buildDefault']);

packages/EasyApiToken/src/Decoders/AbstractApiTokenDecoder.php

@@ -23,24 +23,11 @@ public function __construct(?string $name = null)
 
     public function getName(): string
     {
-        return $this->name ?? self::class;
+        return $this->name ?? static::class;
     }
 
-    /**
-     * @param \Symfony\Component\HttpFoundation\Request|\Psr\Http\Message\ServerRequestInterface $request
-     */
-    protected function getHeaderWithoutPrefix(string $header, string $prefix, $request): ?string
+    protected function getHeaderWithoutPrefix(string $header, string $prefix, Request $request): ?string
     {
-        if ($request instanceof ServerRequestInterface) {
-            @\trigger_error(\sprintf(
-                'Passing $request as %s is deprecated since 2.4 and removed in 3.0. Use %s instead.',
-                ServerRequestInterface::class,
-                Request::class
-            ), \E_USER_DEPRECATED);
-
-            return $this->getHeaderWithoutPrefixForServerRequest($header, $prefix, $request);
-        }
-
         $header = $request->headers->get($header, '');
 
         if (Strings::startsWith($header ?? '', $prefix) === false) {
@@ -63,26 +50,4 @@ protected function getQueryParam(string $param, ServerRequestInterface $request)
 
         return $request->getQueryParams()[$param] ?? null;
     }
-
-    private function getFirstHeaderValue(string $header, ServerRequestInterface $request): ?string
-    {
-        return $request->getHeader(\strtolower($header))[0] ?? null;
-    }
-
-    private function getHeaderWithoutPrefixForServerRequest(
-        string $header,
-        string $prefix,
-        ServerRequestInterface $request
-    ): ?string {
-        if ($this->headerStartsWith($header, $prefix, $request) === false) {
-            return null;
-        }
-
-        return \substr((string)$this->getFirstHeaderValue($header, $request), \strlen($prefix));
-    }
-
-    private function headerStartsWith(string $header, string $prefix, ServerRequestInterface $request): bool
-    {
-        return Strings::startsWith($this->getFirstHeaderValue($header, $request) ?? '', $prefix);
-    }
 }

packages/EasyApiToken/src/Decoders/ApiKeyDecoder.php

@@ -6,13 +6,11 @@
 
 use EonX\EasyApiToken\Interfaces\ApiTokenInterface;
 use EonX\EasyApiToken\Tokens\ApiKey;
+use Symfony\Component\HttpFoundation\Request;
 
 final class ApiKeyDecoder extends AbstractApiTokenDecoder
 {
-    /**
-     * @param \Symfony\Component\HttpFoundation\Request|\Psr\Http\Message\ServerRequestInterface $request
-     */
-    public function decode($request): ?ApiTokenInterface
+    public function decode(Request $request): ?ApiTokenInterface
     {
         $authorization = $this->getHeaderWithoutPrefix('Authorization', 'Basic', $request);
 

packages/EasyApiToken/src/Decoders/BasicAuthDecoder.php

@@ -6,18 +6,11 @@
 
 use EonX\EasyApiToken\Interfaces\ApiTokenInterface;
 use EonX\EasyApiToken\Tokens\BasicAuth;
+use Symfony\Component\HttpFoundation\Request;
 
 final class BasicAuthDecoder extends AbstractApiTokenDecoder
 {
-    public function __construct(?string $name = null)
-    {
-        parent::__construct($name ?? self::NAME_BASIC);
-    }
-
-    /**
-     * @param \Symfony\Component\HttpFoundation\Request|\Psr\Http\Message\ServerRequestInterface $request
-     */
-    public function decode($request): ?ApiTokenInterface
+    public function decode(Request $request): ?ApiTokenInterface
     {
         $authorization = $this->getHeaderWithoutPrefix('Authorization', 'Basic', $request);
 

packages/EasyApiToken/src/Decoders/BearerTokenDecoder.php

@@ -9,6 +9,7 @@
 use EonX\EasyApiToken\Tokens\Jwt;
 use Psr\Log\LoggerInterface;
 use Psr\Log\NullLogger;
+use Symfony\Component\HttpFoundation\Request;
 
 final class BearerTokenDecoder extends AbstractApiTokenDecoder
 {
@@ -30,10 +31,7 @@ public function __construct(JwtDriverInterface $jwtDriver, ?string $name = null,
         parent::__construct($name);
     }
 
-    /**
-     * @param \Symfony\Component\HttpFoundation\Request|\Psr\Http\Message\ServerRequestInterface $request
-     */
-    public function decode($request): ?ApiTokenInterface
+    public function decode(Request $request): ?ApiTokenInterface
     {
         $authorization = $this->getHeaderWithoutPrefix('Authorization', 'Bearer', $request);
 

packages/EasyApiToken/src/Decoders/ChainDecoder.php

@@ -7,6 +7,7 @@
 use EonX\EasyApiToken\Interfaces\ApiTokenDecoderInterface;
 use EonX\EasyApiToken\Interfaces\ApiTokenInterface;
 use EonX\EasyUtils\CollectorHelper;
+use Symfony\Component\HttpFoundation\Request;
 
 final class ChainDecoder extends AbstractApiTokenDecoder
 {
@@ -25,10 +26,7 @@ public function __construct(array $decoders, ?string $name = null)
         parent::__construct($name ?? self::NAME_CHAIN);
     }
 
-    /**
-     * @param \Symfony\Component\HttpFoundation\Request|\Psr\Http\Message\ServerRequestInterface $request
-     */
-    public function decode($request): ?ApiTokenInterface
+    public function decode(Request $request): ?ApiTokenInterface
     {
         foreach ($this->decoders as $decoder) {
             $token = $decoder->decode($request);
@@ -41,5 +39,3 @@ public function decode($request): ?ApiTokenInterface
         return null;
     }
 }
-
-\class_alias(ChainDecoder::class, ChainReturnFirstTokenDecoder::class);