NathanVaughn / webtrees-docker

Up-to-date Docker image for webtrees with all the bells and whistles.
https://hub.docker.com/r/nathanvaughn/webtrees
MIT License
64 stars 16 forks source link

Email not working after webtrees update 2.0.19 > 2.0.21 #88

Closed rubenixnagios closed 2 years ago

rubenixnagios commented 2 years ago

Hello there, first of all thanks so much for this lovely docker image.

Since 2.0.19 update to 2.0.21 email does not work and getting this error message when sending test mail from control panel. Im using gmail account and SMTP options well configured (Port, user & pass)

the update was just pull latest image and run the docker-compose up -d command

Here the error:

Fatal error: Declaration of Egulias\EmailValidator\Validation\MessageIDValidation::isValid(string $email, Egulias\EmailValidator\EmailLexer $emailLexer): bool must be compatible with Egulias\EmailValidator\Validation\EmailValidation::isValid($email, Egulias\EmailValidator\EmailLexer $emailLexer) in /var/www/webtrees/vendor/egulias/email-validator/src/Validation/MessageIDValidation.php on line 23

Same happened a time ago but can't remember what I did to solve it.

Any idea?

Thanks so much in advance.

System: Raspberry pi 4 running raspbian and webtres in a docker container

NathanVaughn commented 2 years ago

I can replicate this too, not sure why is this occurring yet. I'm not much of a PHP person.

rubenixnagios commented 2 years ago

no worries, hope someone finds a solution on this

btw found this over webtrees forum

https://www.webtrees.net/index.php/en/forum/help-for-2-0/35568-sending-email-is-not-working-in-2-0-14

As have it on Docker don't know how to proceed.

Anyways on 2.0.19 docker container it was working. Maybe I try to downgrade to that version...

thanks, let me know if you fins a solution!

rubenixnagios commented 2 years ago

reverted back to 2.0.19 --> I can confirm email is working again.

compared both versions of /var/www/webtrees/vendor directories & are exactly the same! size and files

dunno, I will stay on this version until I find a solution!

will try to compare both /var/www/webtrees versions

NathanVaughn commented 2 years ago

From looking at the webtrees forums, it seems this new file added causes the issue: https://www.webtrees.net/index.php/fr/forum/help-for-2-0/36616-email-error-after-update-to-2-0-21#89985 I can confirm on my Webtrees 2.0.19 container, the file is not there. I'll try to test it and get a patch out today that removes the file in the container.

NathanVaughn commented 2 years ago

I have tested it on my machine and that does appear to fix the issue. Just need to re-push the latest tag now.

NathanVaughn commented 2 years ago

Issue should be resolved, repull the :latest or :2.0.21 tag.

rubenixnagios commented 2 years ago

cool, will try this 2moro that was fast indeed