Bug 16695 – Email checks are not strict enough..

Bugzilla will be upgraded to version 5.0 on a still to be determined date in the near future. The original upgrade date has been delayed.

Bug 16695 - Email checks are not strict enough..

Summary:	Email checks are not strict enough..

Status:	CLOSED CURRENTRELEASE

Aliases:	None

Product:	Red Hat Web Site
Classification:	Red Hat
Component:	Join_Process (Show other bugs)
Sub Component:
Version:	current
Hardware:	i386 Linux

Priority	medium Severity medium
Target Milestone:	---
Target Release:	---
Assigned To:	Tom Lancaster
QA Contact:	Paul Lindner
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2000-08-21 17:23 EDT by Paul Lindner
Modified:	2007-04-18 12:28 EDT (History)
CC List:	0 users

See Also:
Fixed In Version:
Doc Type:	Bug Fix
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2000-08-21 17:24:25 EDT
Type:	---
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Groups: None (edit)

Description Paul Lindner 2000-08-21 17:23:16 EDT

I'm getting a bunch of bounced messages when users enter bad email
addresses.

We should do stricter checking.  Email::Valid supports this.

We need to install Net::DNS, and we need to modify the join callback and
the formmail callback to use the new Email::Valid features..

Comment 1 Paul Lindner 2000-08-21 17:24:22 EDT

Tom, can you have a crack at this.  Should be simple...

Comment 2 Tom Lancaster 2000-08-28 16:32:09 EDT

This is done as requested. The Net::DNS modules are in cvs under lib/
I've implemented the checks by adding the parameter 'mxcheck => 1' to the
Email::Valid->address
call in join/callback.apm.

Please note that I'm not sure how much of a good idea this is, because it
*significantly* slows response time when the user submits.

This is not live yet, just on webdev.

Note You need to log in before you can comment on or make changes to this bug.