-
Notifications
You must be signed in to change notification settings - Fork 46
[WIP] Composite auth adapter #76
base: develop
Are you sure you want to change the base?
[WIP] Composite auth adapter #76
Conversation
@stefanotorresi This looks fantastic! Any chance you can wrap up the lingering todo items soonish? |
@weierophinney been a bit busy lately, but I'll try to attend to this before the end of this week! |
@stefanotorresi I pulled this locally, rebased it off of current develop, and tried to push to your branch... but could not. So I did a PR for you: stefanotorresi#1 The tasks you have listed above are still incomplete, and, honestly, I'm unsure how to complete the first 2. If you would like to continue with this patch, please merge the PR I sent you, and push changes back to your branch when complete. Thanks! |
This repository has been closed and moved to laminas-api-tools/api-tools-mvc-auth; a new issue has been opened at laminas-api-tools/api-tools-mvc-auth#17. |
This repository has been moved to laminas-api-tools/api-tools-mvc-auth. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:
|
As originally proposed in #65
This PR adds the possibility to use multiple authentication adapters under the same
type
.It also introduces an
AdapterAbstractFactory
that allows to retrieve each adapter from theServiceManager
under the naming conventionzf-mvc-auth-authentication-adapters-{type}
.This enables a new method to register custom adapters: using a service name the
adapter
key of a named adapter configuration.TO-DO: