Fix(doc): vulnerability reporting email link#519
Open
alex2wong wants to merge 3 commits intocloudwego:mainfrom
Open
Fix(doc): vulnerability reporting email link#519alex2wong wants to merge 3 commits intocloudwego:mainfrom
alex2wong wants to merge 3 commits intocloudwego:mainfrom
Conversation
Updated the vulnerability reporting email link to include a subject.
|
|
alex2wong
changed the title
mrh997
reviewed
Nov 21, 2025
README.md
Outdated
|
|
||
| If you discover a potential security issue in this project, or think you may | ||
| have discovered a security issue, we ask that you notify Bytedance Security via our [security center](https://security.bytedance.com/src) or [vulnerability reporting email](sec@bytedance.com). | ||
| have discovered a security issue, we ask that you notify Bytedance Security via our [security center](https://security.bytedance.com/src) or [vulnerability reporting email](mailto:sec@bytedance.com?subject=关于Eino的反馈). |
Contributor
There was a problem hiding this comment.
subject 可以改成英文的，另外辛苦把中文的 README 也改一下吧
Update the email link for security reports to include a subject.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated the vulnerability reporting email link to include a subject.
What type of PR is this?
docs: Documentation only changes
Check the PR title.
(Optional) Translate the PR title into Chinese.
修复 vulnerability reporting email 链接为正确的邮箱协议
(Optional) More detailed description for this PR(en: English/zh: Chinese).
en: To correctly define email links in Markdown, the mailto: protocol prefix is required. Otherwise, when rendered as HTML, the link will be treated as a regular URL path (relative to the current page's route), resulting in navigation errors.
The original syntax vulnerability reporting email lacks the mailto: protocol. browsers will interpret it as a relative path combining "the current page's route + sec@bytedance.com", which naturally leads to a 404 page.
The correct approach is to add the mailto: protocol prefix before the email address.
zh(optional): 在 Markdown 中正确定义邮箱链接，需要使用 mailto: 协议前缀，否则渲染为 HTML 时，链接会被当作普通 URL 路径（相对当前页面路由），导致跳转错误。
原来的写法 vulnerability reporting email 缺少 mailto: 协议，浏览器会将其视为 “当前页面路由 + sec@bytedance.com” 的相对路径，自然会找不到页面。
正确写法是在邮箱地址前添加 mailto: 协议前缀
(Optional) Which issue(s) this PR fixes:
(optional) The PR that updates user documentation: