Cast subject to string before passing to SDK

Created on 8 September 2023, 10 months ago

Updated 9 September 2023, 10 months ago

If a markup object is passed for subject, the Assert for subject can blow up memory

  public static function string($value, $property, $message = NULL) {
    if (!is_string($value)) {

Happens with \SendGrid\Mail\Personalization::setSubject

\Drupal\sendgrid_integration\Plugin\Mail\SendGridMail::doMail should cast to string before calling

🐛 Bug report

Status

RTBC

Version

2.0

Component

Code

Created by

🇺🇸United States mglaman WI, USA

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @mglaman
Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update 10 months ago
Waiting for branch to pass
@mglaman opened merge request.
Status changed to Needs review 10 months ago6:56pm 8 September 2023
Comment 10 months ago →
🇺🇸United States mglaman WI, USA
Comment 10 months ago →
🇬🇧United Kingdom c_archer Cumbria
I can confirm making this change has resolved the issue for us.
Status changed to RTBC 10 months ago10:52am 9 September 2023
Comment 10 months ago →
🇬🇧United Kingdom c_archer Cumbria

Production build 0.69.0 2024