[PR #7] [MERGED] Allow email field to be empty in allowedUsers file #54

New issue

Closed

opened 2026-02-26 18:33:11 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 18:33:11 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/decke/smtprelay/pull/7
Author: @JonathonReinhart
Created: 2/9/2021
Status: ✅ Merged
Merged: 2/9/2021
Merged by: @decke

Base: master ← Head: allow-empty-auth-email

📝 Commits (1)

f33105f Allow email field to be empty in allowedUsers file

📊 Changes

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

View changed files

📝 auth.go (+14 -3)
📝 main.go (+1 -1)
📝 smtprelay.ini (+1 -1)

📄 Description

I have a use case where I want to allow a client to send mail as more than just a single address, but still be required to login with username+password.

This PR allows the "email" field of the allowedUsers file (which was luckily the last field) to be missing. In this case, the check in senderChecker is skipped.

_{🔄 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/decke/smtprelay/pull/7 **Author:** [@JonathonReinhart](https://github.com/JonathonReinhart) **Created:** 2/9/2021 **Status:** ✅ Merged **Merged:** 2/9/2021 **Merged by:** [@decke](https://github.com/decke) **Base:** `master` ← **Head:** `allow-empty-auth-email` --- ### 📝 Commits (1) - [`f33105f`](https://github.com/decke/smtprelay/commit/f33105f83c35325decaa31248bb2575ffff3683c) Allow email field to be empty in allowedUsers file ### 📊 Changes **3 files changed** (+16 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `auth.go` (+14 -3) 📝 `main.go` (+1 -1) 📝 `smtprelay.ini` (+1 -1) </details> ### 📄 Description I have a use case where I want to allow a client to send mail as more than just a single address, but still be required to login with username+password. This PR allows the "email" field of the `allowedUsers` file (which was luckily the last field) to be missing. In this case, the check in `senderChecker` is skipped. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>