You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.
Specifically this change breaks laravel/passport ^3.0 which depends on ~1.0 of this library.
I've opened a ticket for them to patch branch 3.x, but that patch may introduce new problems. That's why I'd suggest this update be reverted from 1.x of the zend framework.
The text was updated successfully, but these errors were encountered:
This is problematic to back out of the 1.8 series of zend-diactoros.
The patch that introduced it, #325, did so in order to make the library comply correctly with the PSR-7 specification. Essentially, by not checking it, we had introduced a bug in our implementation.
That also means that, on the flip side, if you were relying on that behavior previously, you were relying on a bug.
If laravel/passport requires the original behavior, one solution is to do the following in your composer.json:
My 2 cents on the issue: I wouldn't revert the change if it was a bug fix to comply to PSR-7 more. Implementing libraries were relying on faulty behavior like @weierophinney noted.
The following commit causes breaking issues and probably should not have been implemented in a minor version.
ee4bcdc#diff-5abca4d9d5693da46d10a54795b1192d
Specifically this change breaks laravel/passport ^3.0 which depends on ~1.0 of this library.
I've opened a ticket for them to patch branch 3.x, but that patch may introduce new problems. That's why I'd suggest this update be reverted from 1.x of the zend framework.
The text was updated successfully, but these errors were encountered: