Fix error reporting in admin and some small fixes #2328

Closed
opened 2026-04-06 03:26:20 +02:00 by MrUnknownDE · 0 comments
Owner

Originally created by @BlackDex on 10/7/2021

  • Fixed a bug in JavaScript which caused no messages to be shown to the
    user in-case of an error send by the server.
  • Changed mail error handling for better error messages
  • Changed user/org actions from a to buttons, this should prevent
    strange issues in-case of javascript issues and the page does re-load.
  • Added Alpine and Debian info for the running docker image

During the mail error testing i encountered a bug which caused lettre to
panic. This panic only happens on debug builds and not release builds,
so no need to update anything on that part. This bug is also already
fixed. See https://github.com/lettre/lettre/issues/678 and https://github.com/lettre/lettre/pull/679

Resolves #2021
Could also fix the issue reported here #2022, or at least no hash # in
the url.

*Originally created by @BlackDex on 10/7/2021* - Fixed a bug in JavaScript which caused no messages to be shown to the user in-case of an error send by the server. - Changed mail error handling for better error messages - Changed user/org actions from a to buttons, this should prevent strange issues in-case of javascript issues and the page does re-load. - Added Alpine and Debian info for the running docker image During the mail error testing i encountered a bug which caused lettre to panic. This panic only happens on debug builds and not release builds, so no need to update anything on that part. This bug is also already fixed. See https://github.com/lettre/lettre/issues/678 and https://github.com/lettre/lettre/pull/679 Resolves #2021 Could also fix the issue reported here #2022, or at least no hash `#` in the url.
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github/vaultwarden#2328