[PR #1049] [MERGED] Use local time in email notifications for new device logins #2868

New issue

Closed

opened 2026-03-03 09:08:42 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 09:08:42 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/1049
Author: @jjlin
Created: 7/8/2020
Status: ✅ Merged
Merged: 7/8/2020
Merged by: @dani-garcia

Base: master ← Head: local-tz

📝 Commits (1)

a28ebcb Use local time in email notifications for new device logins

📊 Changes

4 files changed (+52 additions, -12 deletions)

View changed files

📝 Cargo.lock (+22 -2)
📝 Cargo.toml (+3 -2)
📝 src/api/identity.rs (+5 -3)
📝 src/mail.rs (+22 -5)

📄 Description

In this implementation, the TZ environment variable must be set
in order for the formatted output to use a more user-friendly
time zone abbreviation (e.g., UTC). Otherwise, the output uses
the time zone's UTC offset (e.g., +00:00).

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/dani-garcia/vaultwarden/pull/1049 **Author:** [@jjlin](https://github.com/jjlin) **Created:** 7/8/2020 **Status:** ✅ Merged **Merged:** 7/8/2020 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `master` ← **Head:** `local-tz` --- ### 📝 Commits (1) - [`a28ebcb`](https://github.com/dani-garcia/vaultwarden/commit/a28ebcb401be9a2b0c052d1db319a0c5d6622a3d) Use local time in email notifications for new device logins ### 📊 Changes **4 files changed** (+52 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+22 -2) 📝 `Cargo.toml` (+3 -2) 📝 `src/api/identity.rs` (+5 -3) 📝 `src/mail.rs` (+22 -5) </details> ### 📄 Description In this implementation, the `TZ` environment variable must be set in order for the formatted output to use a more user-friendly time zone abbreviation (e.g., `UTC`). Otherwise, the output uses the time zone's UTC offset (e.g., `+00:00`). --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>