Install page expects an email for "Sender Email". #386

Closed
opened 2025-11-02 03:21:10 -06:00 by GiteaMirror · 3 comments
Owner

Originally created by @puffybsd on GitHub (Feb 23, 2017).

  • Gitea version (or commit ref):
  • Git version: 2.11.1
  • Operating system: Arch Linux
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:

Description

The install page SMTP configuration requires an email address for "Sender Email"; however, many SMTP services such as Spark Post and Send Grid have APIs that require user names that are not email addresses. Configuring the user in app.ini works, but is not user-friendly.

This issue has been fixed in an accompanying pull request.

Screenshots

gitea-sender-email-not-valid-email

If regarding the Web Interface, please include a screenshot

Originally created by @puffybsd on GitHub (Feb 23, 2017). - Gitea version (or commit ref): - Git version: 2.11.1 - Operating system: Arch Linux - Database (use `[x]`): - [ ] PostgreSQL - [x] MySQL - [ ] SQLite - Can you reproduce the bug at https://try.gitea.io: - [ ] Yes (provide example URL) - [ ] No - [x] Not relevant - Log gist: ## Description The install page SMTP configuration requires an email address for "Sender Email"; however, many SMTP services such as Spark Post and Send Grid have APIs that require user names that are not email addresses. Configuring the user in app.ini works, but is not user-friendly. This issue has been fixed in an accompanying pull request. ## Screenshots ![gitea-sender-email-not-valid-email](https://cloud.githubusercontent.com/assets/418252/23239107/4246df00-f933-11e6-852a-2f2a95834aab.png) **If regarding the Web Interface, please include a screenshot**
GiteaMirror added the type/bug label 2025-11-02 03:21:10 -06:00
Author
Owner

@strk commented on GitHub (Feb 23, 2017):

Makes sense to me, LGTM

@strk commented on GitHub (Feb 23, 2017): Makes sense to me, LGTM
Author
Owner

@lunny commented on GitHub (Feb 23, 2017):

@strk This is the issue, not the PR.

@lunny commented on GitHub (Feb 23, 2017): @strk This is the issue, not the PR.
Author
Owner

@strk commented on GitHub (Feb 23, 2017):

Yep, figured after the fact. LGTM'd the PR too...

@strk commented on GitHub (Feb 23, 2017): Yep, figured after the fact. LGTM'd the PR too...
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#386