Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Debug option is nice if you're actively working on tweaking mails or debugging issues. Problem is when Sparkpost fails for some reason (i.e. api.sparkpost.com is temporarily unavailable) and debug setting is off. Proposition is to log message if something went wrong no matter if debug is on or off, and if it's on log message wrapper as well, which provides extensive debug data.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | log_errors_even_if-2778925-4.patch | 1.53 KB | zaporylie |
| #2 | log_errors_even_if-2778925-2.patch | 1.41 KB | zaporylie |
Comments
Comment #2
zaporylieShould be enough
Comment #3
zaporylieOn the other hand maybe I should limit message variables since modules like dblog stores serialized message_variable array in database.
Comment #4
zaporylie#2 with changes from #3
Comment #5
eiriksmWorks as expected. Would be ideal to get a test for this, but seems minor enough to just do later.
Comment #7
zaporylie