Remove PersistenceInterface inheritance from ImpersonationInterface

[segy]

solves #648

[dereuromark]

Instead of changing the existing one, which is also somewhat BC breaking
Would it work to create your own StatelessImpersationInterface extends ImpersonationInterface instead?
And for your case just throw NotImplementedException instead in those methods of the PersistenceInterface?

[dereuromark]

I do agree that

extends AbstractAuthenticator implements PersistenceInterface, ImpersonationInterface

is not clean, though.
As it duplicates the interface contract for the persistence part.

As such, maybe the change is fine after all - given that the changelog could outline the behavior change.

It seems to work because

interface AuthenticationServiceInterface extends PersistenceInterface

so the now "removed" part of Persistence on Impersonation

class AuthenticationService implements AuthenticationServiceInterface, ImpersonationInterface

is covered by that interface.