Opened 16 years ago

Closed 7 years ago

#17043 closed enhancement (worksforme)

trac should check (syntactical) validity of e-mail addresses in tickets

Reported by: vinc17@… Owned by: admin@…
Priority: Low Milestone:
Component: server/hosting Version:
Keywords: Cc: cooljeanius (Eric Gallager), larryv (Lawrence Velázquez)
Port:

Description

trac should check the (syntactical) validity of e-mail addresses in ticket submissions, e.g. in "Owned by" and Cc fields. In practice, it should check that they contain a @ (and only one).

As trac requires full e-mail addresses, contrary to Portfiles, this check would be useful since it is easy to do a mistake.

Change History (8)

comment:1 Changed 16 years ago by blb@…

Owner: changed from jmpp@… to wsiegrist@…

comment:2 Changed 16 years ago by wsiegrist@…

Component: websiteserver/hosting
Keywords: trac removed
Milestone: Website & Documentation
Priority: NormalLow
Status: newassigned
Version: 1.6.0

comment:3 Changed 16 years ago by jmroot (Joshua Root)

When the reassign-to field was first changed to a text box, there was talk of adding code to autocomplete bare usernames by adding @macports.org.

comment:4 Changed 12 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:5 Changed 11 years ago by larryv (Lawrence Velázquez)

Cc: larryv@… added

Cc Me!

comment:6 Changed 10 years ago by jmroot (Joshua Root)

Owner: changed from wsiegrist@… to admin@…
Status: assignednew

comment:7 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)

Owner: changed from admin@… to admin@…
Status: newassigned

comment:8 Changed 7 years ago by raimue (Rainer Müller)

Resolution: worksforme
Status: assignedclosed

After switching to GitHub, the fields can now also take just GitHub user names. Ultimately we cannot validate those, as their might be GitHub users who never logged into Trac. However, we also have auto-completion for known users. I think this is enough validation to avoid obvious mistakes.

Note: See TracTickets for help on using tickets.