[PR #4464] [CLOSED] Allows tls.servername to be set in nodemailer #4258

Closed
opened 2026-04-25 00:19:02 +02:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/advplyr/audiobookshelf/pull/4464
Author: @Vito0912
Created: 7/7/2025
Status: Closed

Base: masterHead: feat/tlsName


📝 Commits (1)

📊 Changes

3 files changed (+47 additions, -5 deletions)

View changed files

📝 client/pages/config/email.vue (+33 -2)
📝 client/strings/en-us.json (+3 -0)
📝 server/objects/settings/EmailSettings.js (+11 -3)

📄 Description

Brief summary

See linked issue in the issue below to understand the fixes better. Adds a field to add validation for a certificate even when using an IP

Which issue is fixed?

Fixes #4462

In-depth Description

How have you tested this?

Not tested yet. This is only from the documentation and should be tested by @scj643

Screenshots

grafik


🔄 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/advplyr/audiobookshelf/pull/4464 **Author:** [@Vito0912](https://github.com/Vito0912) **Created:** 7/7/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `feat/tlsName` --- ### 📝 Commits (1) - [`20ca1b3`](https://github.com/advplyr/audiobookshelf/commit/20ca1b32d78ce330582612f25339c039977e6383) fix ### 📊 Changes **3 files changed** (+47 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `client/pages/config/email.vue` (+33 -2) 📝 `client/strings/en-us.json` (+3 -0) 📝 `server/objects/settings/EmailSettings.js` (+11 -3) </details> ### 📄 Description <!-- For Work In Progress Pull Requests, please use the Draft PR feature, see https://github.blog/2019-02-14-introducing-draft-pull-requests/ for further details. If you do not follow this template, the PR may be closed without review. Please ensure all checks pass. If you are a new contributor, the workflows will need to be manually approved before they run. --> ## Brief summary See linked issue in the issue below to understand the fixes better. Adds a field to add validation for a certificate even when using an IP <!-- Please provide a brief summary of what your PR attempts to achieve. --> ## Which issue is fixed? <!-- Which issue number does this PR fix? Ex: "Fixes #1234" --> Fixes #4462 ## In-depth Description <!-- Describe your solution in more depth. How does it work? Why is this the best solution? Does it solve a problem that affects multiple users or is this an edge case for your setup? --> ## How have you tested this? <!-- Please describe in detail with reproducible steps how you tested your changes. --> Not tested yet. This is only from the documentation and should be tested by @scj643 ## Screenshots <!-- If your PR includes any changes to the web client, please include screenshots or a short video from before and after your changes. --> ![grafik](https://github.com/user-attachments/assets/eba0a872-c7b7-4fbc-befb-458d0286978c) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2026-04-25 00:19:02 +02:00
adam closed this issue 2026-04-25 00:19:02 +02:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/audiobookshelf#4258