Skip to content

Update contributor-guide.md,private email address securization#1492

Closed
momomtf wants to merge 2 commits into
project-trans:masterfrom
momomtf:patch-8
Closed

Update contributor-guide.md,private email address securization#1492
momomtf wants to merge 2 commits into
project-trans:masterfrom
momomtf:patch-8

Conversation

@momomtf
Copy link
Copy Markdown
Contributor

@momomtf momomtf commented Jul 13, 2025

增加一处提醒,提醒贡献者自己注册github的电邮有可能会被公开,以及技术手段以预防这一点。

Signed-off-by: Momo <189544439+momomtf@users.noreply.github.com>
Copy link
Copy Markdown

@project-trans-bot project-trans-bot Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 预览部署完成!访问链接: https://53d6d704.mtfwiki-preview.pages.dev

✨ 本 PR 修改了以下页面: ✨
https://53d6d704.mtfwiki-preview.pages.dev/zh-cn/docs/contributor-guide

Copy link
Copy Markdown
Contributor

@AnnaBeiyan AnnaBeiyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不建议在序号列表中插入这么长内容的 warning,非常影响阅读体验。另外自己通过GitHub贡献本来就有很多方面均有可能泄露个人信息,包括电子邮件地址和exif信息等,建议简要一句话叙述即可。相比exif信息泄露,电子邮件地址泄露的后果一般要轻微的多。另外需要说明的是,由于Git的分布式特性,commit中署名的电子邮件地址未必是由署名的作者所提交的。

Signed-off-by: Momo <189544439+momomtf@users.noreply.github.com>
@momomtf
Copy link
Copy Markdown
Contributor Author

momomtf commented Jul 14, 2025

不建议在序号列表中插入这么长内容的 warning,非常影响阅读体验。另外自己通过GitHub贡献本来就有很多方面均有可能泄露个人信息,包括电子邮件地址和exif信息等,建议简要一句话叙述即可。相比exif信息泄露,电子邮件地址泄露的后果一般要轻微的多。另外需要说明的是,由于Git的分布式特性,commit中署名的电子邮件地址未必是由署名的作者所提交的。

已修改,感谢反馈

@Leetfs Leetfs requested a review from AnnaBeiyan July 20, 2025 15:17
@AnnaBeiyan
Copy link
Copy Markdown
Contributor

其实我有点想拒绝这个 PR,我觉得这句话应该已经够了:“该方法的过程将是公开的,请注意处理隐私数据,如果不会处理隐私数据,请通过邮件方式贡献内容”。而且 GitHub 注册账号在线编辑会默认公开电子邮件吗?这个还需要确认一下。

@saeziae
Copy link
Copy Markdown
Member

saeziae commented Jul 21, 2025

提醒git記錄有電郵還是有必要的吧,而且github也有一鍵隱藏的選項可供使用

## 贡献方式 {#method}

1. 会 GitHub 操作,直接 <abbr title="Pull Request">PR</abbr> 或 通过 {{< shields/github-issue "project-trans/mtf-wiki" >}}(该方法的过程将是公开的,请注意处理隐私数据,如果不会处理隐私数据,请通过邮件方式贡献内容)
{{< notice warning "注意事项" >}}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
{{< notice warning "注意事项" >}}

1. 会 GitHub 操作,直接 <abbr title="Pull Request">PR</abbr> 或 通过 {{< shields/github-issue "project-trans/mtf-wiki" >}}(该方法的过程将是公开的,请注意处理隐私数据,如果不会处理隐私数据,请通过邮件方式贡献内容)
{{< notice warning "注意事项" >}}
如果你在Github网页上编辑,Github默认会把你注册用的电邮地址放在你编辑的内容上的“commit message”里面;一旦编辑完并提交了“pull request”请求,**所有人都可以看得到这个地址**
{{< /notice >}}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
{{< /notice >}}

@Cryolitia Cryolitia added the Waiting | 等待作者回复 等待作者回复其他参与者的评论或修改建议 label Aug 17, 2025
@yingxue03
Copy link
Copy Markdown
Contributor

而且 GitHub 注册账号在线编辑会默认公开电子邮件吗?这个还需要确认一下。

“Keep my email address private”选项默认是不开启的

https://docs.github.com/en/account-and-profile/concepts/email-addresses#commit-email-addresses


1. 会 GitHub 操作,直接 <abbr title="Pull Request">PR</abbr> 或 通过 {{< shields/github-issue "project-trans/mtf-wiki" >}}(该方法的过程将是公开的,请注意处理隐私数据,如果不会处理隐私数据,请通过邮件方式贡献内容)
{{< notice warning "注意事项" >}}
如果你在Github网页上编辑,Github默认会把你注册用的电邮地址放在你编辑的内容上的“commit message”里面;一旦编辑完并提交了“pull request”请求,**所有人都可以看得到这个地址**
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

如果你在Github网页上编辑,Github默认会把你注册用的电邮地址放在你编辑的内容上的“commit message”里面

不准确,电邮地址并不仅仅出现在commit message中

@yingxue03
Copy link
Copy Markdown
Contributor

只提及“请注意处理隐私数据”有可能让贡献者只留意到自己所知的的隐私数据,而电邮地址会被公开记录且极难清除这一点新手可能并不了解(比如说pr目前的内容仍然不够准确……)。

@saeziae
Copy link
Copy Markdown
Member

saeziae commented Feb 15, 2026

“Keep my email address private”选项默认是不开启的

增加内容告知贡献者有这个选项。

@Cryolitia
Copy link
Copy Markdown
Member

Cryolitia commented Mar 25, 2026

instead by #1650 感谢贡献

@Cryolitia Cryolitia closed this Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content | 内容 Waiting | 等待作者回复 等待作者回复其他参与者的评论或修改建议

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants