Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Split middleware support into satellite package and deprecate MiddlewareListener #8

Closed
weierophinney opened this issue Dec 31, 2019 · 4 comments · Fixed by #51
Closed

Comments

@weierophinney
Copy link
Member

Number of issues arose from version incompatibilities due to lack of version constraints on zend-stratigility.

A way forward is to deprecate Middleware listener and introduce satellite package defining dependencies both on zend-mvc and zend-stratigility.

zendframework/zend-mvc-middleware#1


Originally posted by @Xerkus at zendframework/zend-mvc#308

@weierophinney
Copy link
Member Author

In the absence of this being split out yet, are the correct versions for these dependencies documented
for zend-mvc 3.1 + zend-stratigility?


Originally posted by @christaggart at zendframework/zend-mvc#308 (comment)

@weierophinney
Copy link
Member Author

@christaggart we have the following constraints in our project:

"zendframework/zend-mvc": "^3.1",
"zendframework/zend-stratigility": "<2.2",

that currently installs zend-mvc: 3.1.1 and zend-stratigility: 2.1.2

Afaik, zend-stratigility 2.2 and 3.x don't work with the current zend-mvc.


Originally posted by @rieschl at zendframework/zend-mvc#308 (comment)

@weierophinney
Copy link
Member Author

BTW: I just saw that the composer.json of the current master also suggests that.


Originally posted by @rieschl at zendframework/zend-mvc#308 (comment)

@weierophinney
Copy link
Member Author

Hi guys some status? now im using zend events for middleware render next step, thats very ugly


Originally posted by @italodeveloper at zendframework/zend-mvc#308 (comment)

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 a pull request may close this issue.

2 participants