Opened 11 years ago

Closed 6 years ago

#404 closed bug (fixed)

Editmyprofile and may be to adminmandatory allows to enter invalid email

Reported by: jeanyves Owned by: jeanyves
Priority: major Milestone: Legacy Resolved
Component: BW Profile Keywords: EditMyProfile Email
Cc:

Description

The consequence is that mailbot failed

in addition editmyprofile is to use NewReplaceInmemberstrads? instead of ReplaceInmemberstrads?

Change History (13)

comment:1 Changed 11 years ago by lemon-head

In my opinion, the email and pw setting belongs into mypreferences, not into editmyprofile!

comment:2 Changed 11 years ago by feuerdaemon

  • Component changed from unknown to BW Mail
  • Milestone changed from unassigned to BigPicture

comment:3 Changed 11 years ago by philipp

  • Milestone changed from BigPicture to unassigned

Milestone BigPicture? deleted

comment:4 Changed 7 years ago by micha

  • Component changed from BW Mail to BW Profile
  • Milestone changed from unassigned to 0.5.8 - bugfixing

comment:5 Changed 7 years ago by planetcruiser

micha, do you commit to doing this within the next 2 weeks until the milestone is released? if not, please assign it to the next milestone or set it to milestone unassigned, because 0.5.8 is a bit crowded. 5 tickets per milestone has proven to be a realistic aim at current pace.

comment:6 Changed 7 years ago by planetcruiser

  • Milestone changed from 0.5.8 - bugfixing to unassigned

micha is on maternity leave :)

comment:7 Changed 6 years ago by TimLoal

  • Milestone changed from Future to Legacy Resolved
  • Resolution set to fixed
  • Status changed from new to closed

comment:8 Changed 6 years ago by planetcruiser

  • Resolution fixed deleted
  • Status changed from closed to reopened

micha never committed anything. could someone check if the bug still occurs?

comment:9 Changed 6 years ago by shevek

Actually the mail check that is implemented on editmyprofile is to strict as well. So "shevekÄÖÜ"@blafaselblubb.de would be a valid email address but isn't accepted. And shevek@müller.at isn't accepted either.

comment:10 Changed 6 years ago by shevek

Additionally the JS used to check the email address uses a different regexp to determine if the address is a valid one than the backend code (which seems to be the one from RFC 2822).

comment:11 Changed 6 years ago by globetrotter_tt

only for the record: adminmandatory is not used anymore

comment:12 Changed 6 years ago by shevek

Would be interesting which invalid email addresses could be entered. Because now there are checks to avoid this I believe it's likely this is fixed anyhow.

comment:13 Changed 6 years ago by planetcruiser

  • Resolution set to fixed
  • Status changed from reopened to closed

i trust your word on it. :)

Note: See TracTickets for help on using tickets.