PostLoginAccountHandler::handle encounters a failed assertion

[alihammad-gist]

Bug Report

Get assertion error after login.

AssertionError (1)
assert($this->authenticationService instanceof AuthenticationServiceInterface)

Q	A
Version(s)	7

Summary

After login the handler PostLoginAccountHandler::handle encounters a failed assertion. The assertion asserts that Laminas\Authentication\AuthenticationService is an instance of Core\App\Service\AuthenticationServiceInterface .

Current behavior

AssertionError is thrown

How to reproduce

1. Login with valid credentials.

Expected behavior

Being able to see the admin dashboard after login.

[alexmerlin]

Hi @alihammad-gist
Thank you for opening an issue + PR - we appreciate the effort.

Unfortunately, we cannot confirm the issue.
Could you provide more information about your server setup? PHP version/engine, platform (WSL/Docker) etc...

Note that the demo deployment on https://admin7.dotkernel.net/ is running the latest code currently available on the 7.0 branch - including the code you mentioned not working correctly.

[alihammad-gist]

@alexmerlin I am using php 8.4.12 with nginx, php-fpm & mariadb. Maybe the demo deployment may have assertions disabled for production? Just guessing. I have attached a short video for demonstration.

https://youtu.be/2NHB1aPtkxc

[alexmerlin]

@alihammad-gist Thanks for the update (and even making a video demo with your issue). 👍

Te purpose of Core\App\Service\AuthenticationServiceInterface is to avoid these errors falsly reported by PHPStan:

 ------ -------------------------------------------------------------------------------------------------- 
  Line   src/Admin/src/Handler/Account/PostLoginAccountHandler.php                                         
 ------ -------------------------------------------------------------------------------------------------- 
  83     Call to an undefined method Laminas\Authentication\AuthenticationServiceInterface::getAdapter().  
         🪪  method.notFound                                                                               
         at src/Admin/src/Handler/Account/PostLoginAccountHandler.php:83                                   
  108    Call to an undefined method Laminas\Authentication\AuthenticationServiceInterface::getStorage().  
         🪪  method.notFound                                                                               
         at src/Admin/src/Handler/Account/PostLoginAccountHandler.php:108                                  
 ------ --------------------------------------------------------------------------------------------------

because both getAdapter and getStorage exist in vendor/laminas/laminas-authentication/src/AuthenticationService.php but not in vendor/laminas/laminas-authentication/src/AuthenticationServiceInterface.php.

---

One solution would be to use the concrete implementation of AuthenticationService, by replacing:

protected \Laminas\Authentication\LaminasAuthenticationServiceInterface $authenticationService,

with

protected \Laminas\Authentication\AuthenticationService $authenticationService,

but that beats the purpose of the whole idea of using interfaces for overwritability/testability and replacing one implementation of AuthenticationServiceInterface with another one.