No confirmation email sent to user after a successful password reset
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Tranquilla
- Stack tecnologico
- typescript
- Ambito
- authentication
Direzione di ricerca
Leggi application/backend/src/controllers/UsersController.ts, concentrandoti su resetPassword e su come generatePasswordResetLink invia le e-mail. Esamina i modelli esistenti per l’invio di e-mail e i test pertinenti prima di apportare la modifica. Il lavoro è completato quando un reset riuscito invia una conferma all’indirizzo e-mail attuale dell’utente e i test coprono questo comportamento.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
UsersController.resetPassword in application/backend/src/controllers/UsersController.ts validates the reset token, updates the user's password hash, and marks the token as used. but no confirmation email goes out to the user after the reset completes. generatePasswordResetLink sends the initial reset link, but once the user clicks the link and submits a new password, nothing lands in their inbox.
Risk: the user has no way to detect a password reset that wasn't triggered by them. If someone managed to trigger a reset on their account (e.g. by intercepting an already-issued reset link etc), the legitimate user gets no signal until they try to log in with their old password and fail.
Google, Auth0 and most modern SaaS platforms send a "your password was changed" email on every successful reset specifically to catch this class of attack early.
Suggested fix: at the end of resetPassword, after the password hash is written and the token is marked used, send a confirmation email to the user's current email address ("Your CTRL password was changed. If this wasn't you, contact support.").
- Lingua principale
- TypeScript
- Stelle
- 4
- Fork
- 0
- Merge medio
- 14h 50m
- PR unite (30g)
- 5
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Garvan-Data-Science-Platform/ctrl
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 82/100
-
admin-client backend bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
backend bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Garvan-Data-Science-Platform/ctrl#970 · 1 commento ·
-
backend bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
backend bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Tutte le issue di Garvan-Data-Science-Platform/ctrl
Issue simili
-
Create/import wallet form shows raw "password must be at least 1 characters" instead of the translated messageForse già presa @CapThunder19 l’ha presa oggi. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
joinmarket-webui/jam#1538 ·
I maintainer di solito rispondono entro 1 giorno
-
t0ggles
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
nolimits4web/swiper#8240 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
modelcontextprotocol/inspector#2723 ·
I maintainer di solito rispondono entro 1 giorno
-
status:approved type:bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 70/100
egdev6/quorb#109 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno