Skip to content

feat/lazy middleware resolution#2

Open
ralflang wants to merge 6 commits into
FRAMEWORK_6_0from
feat/lazy-middleware-resolution
Open

feat/lazy middleware resolution#2
ralflang wants to merge 6 commits into
FRAMEWORK_6_0from
feat/lazy-middleware-resolution

Conversation

@ralflang
Copy link
Copy Markdown
Member

  • style: php-cs-fixer
  • chore: Bump minimum php version to 8.1 because we want object instantiation in constructors
  • feat: Default constructor arguments hint at horde homegrown implementations
  • feat(handler): lazy middleware resolution via PSR-11 container
  • test: Add middleware and payload handler tests
  • chore: Add optional dependency on psr/container

Add optional ContainerInterface as last constructor parameter.
The addMiddleware/setPayloadHandler methods now accept string class names.
nextMiddleware() resolves strings via container on demand if set.
New resolvePayloadHandler() for lazy controller resolution.
Add psr/container dependency.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant