mirror of
https://github.com/lldap/lldap.git
synced 2026-04-25 00:05:50 +03:00
[PR #453] [MERGED] Improve reset email errors #753
Labels
No labels
backend
blocked
bug
cleanup
dependencies
docker
documentation
duplicate
enhancement
enhancement
frontend
github_actions
good first issue
help wanted
help wanted
integration
invalid
ldap
pull-request
question
rust
rust
tests
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/lldap-lldap#753
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/lldap/lldap/pull/453
Author: @nitnelave
Created: 2/15/2023
Status: ✅ Merged
Merged: 2/15/2023
Merged by: @nitnelave
Base:
main← Head:email📝 Commits (2)
a41d7a3server: Remove trailing / from the domain URL837b574app: improve error message for wrong/expired reset token📊 Changes
5 files changed (+81 additions, -70 deletions)
View changed files
📝
app/src/components/app.rs(+57 -63)📝
app/src/components/reset_password_step2.rs(+12 -4)📝
server/src/infra/auth_service.rs(+6 -2)📝
server/src/infra/mail.rs(+3 -1)📝
server/src/infra/tcp_server.rs(+3 -0)📄 Description
This makes the configuration more robust to a trailing slash in the domain, and improves the error messages when clicking on a non-existing or expired token.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.