Problem/Motivation

I have this problem after submitting the form using Webfom with Emails / Handlers.

Uncaught PHP Exception Drupal\\Core\\Entity\\EntityStorageException: "Invalid address: (Reply-To): " at /web/core/lib/Drupal/Core/Entity/Sql/SqlContentEntityStorage.php line 815,

Steps to reproduce

1- Install smtp and webform
2- After Enabling their modules create an exam form with Emails / Handlers and the value of the Reply-to email is Default.

Proposed resolution

Check if the Reply-To is not empty.

CommentFileSizeAuthor
#2 invalid-address-reply-To-3366074-2.patch699 bytesabdulaziz zaid

Issue fork smtp-3366074

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

abdulaziz zaid created an issue. See original summary.

abdulaziz zaid’s picture

abdulaziz zaid
Primary language Arabic
Location Riyadh 🇸🇦
commented
StatusFileSize
new invalid-address-reply-To-3366074-2.patch699 bytes
abdulaziz zaid’s picture

abdulaziz zaid
Primary language Arabic
Location Riyadh 🇸🇦
commented
Issue summary: View changes
abdulaziz zaid’s picture

abdulaziz zaid
Primary language Arabic
Location Riyadh 🇸🇦
commented
Assigned: Unassigned » abdulaziz zaid
Status: Active » Needs review
japerry’s picture

japerry
Location KOMK
commented
Status: Needs review » Closed (duplicate)

Duplicate of #3323386: Error sending e-mail: Invalid address

stephaneq’s picture

stephaneq
Primary language French
Location Paris 🇫🇷
commented
Assigned: abdulaziz zaid » Unassigned
Status: Closed (duplicate) » Reviewed & tested by the community

The linked issue is the same type of error, but not the exact same use case, as it only focuses on the return-path header. Here the problem is with the reply-to header, so that also needs to be fixed.
I had this issue and the patch provided by Abdulaziz seems to fix it.

dillix’s picture

dillix commented
Priority: Normal » Critical

#2 fixed my webform which sends by smtp module. +1 for RTBC.

bluegeek9’s picture

bluegeek9 commented
Assigned: Unassigned » bluegeek9

bluegeek9 opened merge request !62

bluegeek9’s picture

bluegeek9 commented
Assigned: bluegeek9 » Unassigned
Status: Reviewed & tested by the community » Fixed
//www.flaticon.com/free-icons/thank-you Thank you for your contribution! Your continued support makes this project sustainable.
There are multiple ways to show appreciation for the work contributed to this project including:
  • Triage issues and adding more context to existing issues.
  • Flagging SMTP as a favorite on the project page to help others discover it and show your support.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

dillix’s picture

dillix commented

@bluegeek9 can you release new version of smtp with this fix.

bluegeek9’s picture

bluegeek9 commented

No. I can merge code, but I cannot make new releases.

I am working with another maintainer to review #3195287: Config form: Display override values

We should have a new release shortly after, 3195287.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.