[Privacy] Gitea leaks hidden email addresses #2040

Closed
opened 2025-11-02 04:22:02 -06:00 by GiteaMirror · 4 comments
Owner

Originally created by @ghost on GitHub (Jul 10, 2018).

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

Description

Screenshots

Originally created by @ghost on GitHub (Jul 10, 2018). <!-- 1. Please speak English, this is the language all of us can speak and write. 2. Please ask questions or configuration/deploy problems on our Discord server (https://discord.gg/NsatcWJ) or forum (https://discourse.gitea.io). 3. Please take a moment to check that your issue doesn't already exist. 4. Please give all relevant information below for bug reports, because incomplete details will be handled as an invalid report. --> - Gitea version (or commit ref): 3e445cc - Git version: not relevant - Operating system: not relevant - Database (use `[x]`): - [ ] PostgreSQL - [ ] MySQL - [ ] MSSQL - [ ] SQLite - Can you reproduce the bug at https://try.gitea.io: - [x] Yes (provide example URL) - [ ] No - [ ] Not relevant - Log gist: ## Description ## Screenshots <!-- **If this issue involves the Web Interface, please include a screenshot** -->
GiteaMirror added the topic/security label 2025-11-02 04:22:02 -06:00
Author
Owner

@msg7086 commented on GitHub (Aug 8, 2018):

Just hit this bug due to the CSRF issue. Got a long list of Email addresses for every victim of that issue.

(Automatically followed OP, then got notifications of all events, leaking the Email addresses.)

email-leaking

@msg7086 commented on GitHub (Aug 8, 2018): Just hit this bug due to the CSRF issue. Got a long list of Email addresses for every victim of that issue. (Automatically followed OP, then got notifications of all events, leaking the Email addresses.) ![email-leaking](https://user-images.githubusercontent.com/206155/43812115-b50fe426-9a8d-11e8-9575-ddcc66fd5093.png)
Author
Owner

@techknowlogick commented on GitHub (Aug 10, 2018):

For anyone looking to solve this issue, the line you can change is here: 95f2e2b57b/models/issue_mail.go (L102) it passes all emails into the function, but what could be done is to loop over emails and then individually pass them into the SendIssueMentionMail function (the last argument in that function is where the emails get passed in)

@techknowlogick commented on GitHub (Aug 10, 2018): For anyone looking to solve this issue, the line you can change is here: https://github.com/go-gitea/gitea/blob/95f2e2b57beedcdeb2b9623dc86e26f252fdd7bd/models/issue_mail.go#L102 it passes all emails into the function, but what could be done is to loop over emails and then individually pass them into the `SendIssueMentionMail` function (the last argument in that function is where the emails get passed in)
Author
Owner

@HenrikBengtsson commented on GitHub (Aug 30, 2018):

Not a complaint, just asking for clarification: This fix means that email notifications for issues will be sent out to each watcher independently starting with Gitea 1.5.1, correct? This means that users will no longer be able to do 'Reply All' to continue an email-only thread on the topic, correct?

@HenrikBengtsson commented on GitHub (Aug 30, 2018): Not a complaint, just asking for clarification: This fix means that email notifications for issues will be sent out to each watcher independently starting with Gitea 1.5.1, correct? This means that users will no longer be able to do 'Reply All' to continue an email-only thread on the topic, correct?
Author
Owner

@lafriks commented on GitHub (Aug 30, 2018):

@HenrikBengtsson yes

@lafriks commented on GitHub (Aug 30, 2018): @HenrikBengtsson yes
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#2040