Skip to content

Commit 05cbd2a

Browse files
author
Elliot Smith
committed
Comment out currently-unused class
Once the issue with slim-jwt-auth is fixed (see tuupola/slim-jwt-auth#217), we can reinstate that library as the JWT solution as follows: * Add tuupola/slim-jwt-auth back into composer.json * Remove tuupola/http-factory and tuupola/callable-handler (these are dependencies of tuupola/slim-jwt-auth and will be installed by that library; we no longer need to reference them directly once slim-jwt-auth is back in place) * Uncomment body of App\Middleware\Session\JwtAuthenticationFactory class * Remove App\Middleware\Session\JwtMiddleware file * Remove App\Middleware\Session\JwtMiddlewareFactory file * Uncomment lines 67-68 and remove lines 69-70 of ConfigProvider.php For more context, see commit 3efaa3b in this repo.
1 parent 9c8bc49 commit 05cbd2a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

service-admin/src/App/src/Middleware/Session/JwtAuthenticationFactory.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ class JwtAuthenticationFactory
1919
* @param ContainerInterface $container
2020
* @return JwtAuthentication
2121
*/
22-
public function __invoke(ContainerInterface $container)
22+
/*public function __invoke(ContainerInterface $container)
2323
{
2424
$jwtConfig = $container->get('config')['jwt'];
2525
@@ -45,5 +45,5 @@ public function __invoke(ContainerInterface $container)
4545
];
4646
4747
return new JwtAuthentication(array_merge($jwtConfig, $jwtHandlers));
48-
}
48+
}*/
4949
}

0 commit comments

Comments
 (0)