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
docs: add how to config egg behind a proxy #3614
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
竟然还能攻击……
exports.ipHeaders = 'X-Real-Ip, X-Forwarded-For'; | ||
``` | ||
|
||
### `config.maxProxyCount` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这些配置是不是考虑加载一个 config.xx 上,全挂到根下,回头不好管理,和用户的冲突啥的
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
下个大版本可以统一整理一下
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
感觉可以考虑一波大版本了:
- 单进程
- config 异步
- 一些历史包袱抛掉
@@ -0,0 +1,72 @@ | |||
title: 前置代理模式 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
感觉可以放到安全里面了
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个还是单独放一章吧,很多内容是配置反向代理的,不算安全相关
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
嗯,可以是单独文章,不过回头重构的文档里面我放到安全目录,而不是在教程目录。新文档里面的安全是拆成多个文章了。
这里可以先这样。
docs: add how to config egg behind a proxy (#3614)
Checklist
npm test
passesAffected core subsystem(s)
Description of change