search

getgrav / grav-plugin-email

Grav Email Plugin
http://getgrav.org
MIT License
37 stars 29 forks source link

Update swiftmailer to 5.4.3 #37

Closed Pls closed 8 years ago

Pls commented 8 years ago

E_WARNING - getmypid() has been disabled for security reasons

Getting this warning right now because 5.4.2 version has this function included. Latest version doesn't have it.

Let's update to 5.4.3 version. Cheers :)

rhukster commented 8 years ago

https://github.com/getgrav/grav-plugin-email/commit/c9d7005db379804adeed9915c22531bcc0a03387

Done and released for next GPM update (20 mins)

Pls commented 8 years ago

Cheers!

yaazkal commented 7 years ago

Hi, I have latest verion of the mail plugin and I can see that it has swiftmailer 5.4.3, but this version also has a call to that getmypid() function at lib/classes/Swift/Mime/SimpleMimeEntity.php line 83. So I'm getting a Grav error

Whoops\Exception\ErrorException thrown with message "getmypid() has been disabled for security reasons"

Any fix to that? Do I need to allow that function on the server?

Pls commented 7 years ago

As a workaround for now I replaced all getmypid() function calls with uniqid() calls. That works for me as temporary solution.

flaviocopes commented 7 years ago

Yes that must be enabled on the server, it's an upstream issue. @yaazkal commented in https://github.com/swiftmailer/swiftmailer/issues/745 👍

We can't do anything on this side until they release an update with a fix for this.

yaazkal commented 7 years ago

Thanks @Pls I've replaced the other two references to that function and site is working now; I hope they will include a fix for the next version.

Regards.