No Email Validation When Incorrect Email is Entered while Testing Configuration

Created on 24 November 2023, about 1 year ago
Updated 7 December 2023, about 1 year ago

Problem/Motivation

No email validation is present when an incorrect email is entered in the Recipient field while testing configuration.

Steps to reproduce

  • Navigate to: /admin/config/system/phpmailer-smtp
  • Enter invalid email under Test configuration in Recipient field.
  • The form will be submitted without throwing any error message.

Proposed resolution

Add a Validation, when the Recipient field is not empty.

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Closed: won't fix

Version

2.2

Component

Code

Created by

🇮🇳India keshav patel

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @keshav patel
  • Status changed to Needs review about 1 year ago
  • 🇮🇳India keshav patel

    The provided patch file "fix_mail_validation-3403990-2.patch" adds the email validation when the Recipient field is not empty and will throw an error message for invalid field value.

  • Status changed to Postponed: needs info about 1 year ago
  • 🇦🇺Australia imclean Tasmania

    Thanks for the bug report. The test email field is an HTML field of type "email" which does its own validation. It does allow some unconventional email addresses, such as me@localhost, which can be a valid testing email address. It's a bit loose in some ways and a bit tight in others but I don't think adding extra validation here is necessary.

    I'd also prefer not to add a regex to this module, aside from what the PHPMailer library provides.

    For more information: https://html.spec.whatwg.org/multipage/input.html#valid-e-mail-address

  • Status changed to Closed: won't fix about 1 year ago
  • 🇦🇺Australia imclean Tasmania
Production build 0.71.5 2024