Skip to content
This repository has been archived by the owner on Apr 16, 2024. It is now read-only.

🐛 BUG: Validate password reset email #597

Closed
1 of 2 tasks
PatrickSkowronek opened this issue Apr 9, 2018 · 0 comments · Fixed by #637
Closed
1 of 2 tasks

🐛 BUG: Validate password reset email #597

PatrickSkowronek opened this issue Apr 9, 2018 · 0 comments · Fixed by #637
Assignees
@lukas-schardt
Labels
api All Backend related Issues bug This Issue describes a unwanted behavior good first issue For a new Member web-frontend All frontend related issues

Comments

@PatrickSkowronek
Copy link
Collaborator

PatrickSkowronek commented Apr 9, 2018 •
edited
Loading

Currently we do not validate the e-mail in the frontend and backend.

  • Add empty filter, so nobody can send an empty e-mail address
  • Add e-mail filter set in config (Optimal)
@PatrickSkowronek PatrickSkowronek added bug This Issue describes a unwanted behavior api All Backend related Issues web-frontend All frontend related issues good first issue For a new Member labels Apr 9, 2018
@aeimer aeimer mentioned this issue Apr 9, 2018
Closed
1 task
@lukas-schardt lukas-schardt self-assigned this Apr 12, 2018
lukas-schardt added a commit that referenced this issue Apr 18, 2018
@lukas-schardt
#597: added email validator to reset form, show hint if the email isn…
9f5b80f 
…'t validated correctly
@lukas-schardt lukas-schardt mentioned this issue Apr 18, 2018
Merged
PatrickSkowronek added a commit that referenced this issue Apr 18, 2018
@PatrickSkowronek
Merge branch 'develop' into bugfix/#597-validate-password-reset-email
97e9636
lukas-schardt added a commit that referenced this issue Apr 18, 2018
@lukas-schardt
#597: built a custom email validator, which also filters out emails t…
e0bd29f 
…hat have only a tld/don't have a tld
lukas-schardt added a commit that referenced this issue Apr 19, 2018
@lukas-schardt
#597: added regex source
48a7c90
PatrickSkowronek added a commit that referenced this issue Apr 19, 2018
@PatrickSkowronek
Merge branch 'develop' into bugfix/#597-validate-password-reset-email
9c9e9a4
PatrickSkowronek added a commit that referenced this issue Apr 19, 2018
@PatrickSkowronek
Merge pull request #637 from h-da/bugfix/#597-validate-password-reset…
7a0f180 
…-email

 #597: added email validator to reset form, show hint if the email isn…
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@lukas-schardt lukas-schardt

Labels
api All Backend related Issues bug This Issue describes a unwanted behavior good first issue For a new Member web-frontend All frontend related issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#597: added email validator to reset form, show hint if the email isn… geli-lms/geli
2 participants
@lukas-schardt @PatrickSkowronek