Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update pr#1086 pr#1088 #36

Merged
merged 1 commit into from
Aug 27, 2018
Merged

Update pr#1086 pr#1088 #36

merged 1 commit into from
Aug 27, 2018

Conversation

fralonra
Copy link
Member

No description provided.

@fralonra fralonra requested a review from vincent178 August 25, 2018 16:06
docs/Factory.md Outdated
<a name="factory-trust-proxy"></a>
### `trustProxy`

通过开启 `trustProxy` 选项,Fastify 会明白使用了代理服务,且 `X-Forwarded-*` header 是可信的,否则该值被认为是极具欺骗性的。
Copy link
Contributor

Choose a reason for hiding this comment

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

Fastify 会明白使用了代理服务 这句话改为 Fastify 会认为使用了代理服务,是不是更好一点

Copy link
Member Author

Choose a reason for hiding this comment

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

已修改

@fralonra fralonra mentioned this pull request Aug 26, 2018
21 tasks
@vincent178 vincent178 merged commit fcc8c08 into fastify:master Aug 27, 2018
@fralonra fralonra deleted the pr#1088 branch August 27, 2018 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants