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

Switch to use laminas/laminas-mail instead of zendframework/zend-mail #876

Merged
merged 4 commits into from
Jul 26, 2020

Conversation

glensc
Copy link
Member

@glensc glensc commented Jun 27, 2020

https://github.com/zendframework/zend-mail repository was abandoned on 2019-12-31

Previously we maintained zend-mail fork, because upstream was unacceptably slow to accept pull requests and make a release:

there's 2.10.1 released for laminas-mail that still does not contain the fixes:

  • #83 fixes PHPDocs in Transport\InMemory (the last message can be null).
  • #84 fixes PHP 7.4 compatibility.
  • #82 fixes numerous issues in Storage\Maildir. This storage adapter was not working before and unit tests were disabled.
  • #75 fixes how Laminas\Mail\Header\ListParser::parse() parses the string with quotes.
  • #88 fixes recognizing encoding of Subject and GenericHeader headers.

the version we used was branched of 2.10.0, with following backports:

@glensc glensc added this to the 3.9.0 milestone Jun 27, 2020
@glensc glensc self-assigned this Jun 27, 2020
@glensc glensc marked this pull request as draft June 27, 2020 08:04
@glensc glensc marked this pull request as ready for review June 30, 2020 21:10
@glensc glensc modified the milestones: 3.9.0, 3.9.1 Jul 1, 2020
@glensc glensc force-pushed the laminas-mail branch 2 times, most recently from 7bf6550 to 2df263a Compare July 12, 2020 10:12
@glensc glensc modified the milestones: 3.9.1, 3.9.2 Jul 13, 2020
@glensc glensc merged commit 8cc11a0 into eventum:master Jul 26, 2020
@glensc glensc deleted the laminas-mail branch July 26, 2020 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant