Open
Conversation
Merge from trunk
Merge from trunk
Contributor
Author
|
另外我重置了一个版本:https://github.com/funny/gateway 有以下差异:
对于第三点还不是最终确认版本,我可能会加一个后端主动连接网关的协议,网关主动连接后端时不发送客户端地址,后端主动连接网关的时候才发送客户端地址,这样可以保持后端的最小化改造。 以上修改可以作为参考。 |
|
9 年后,关于服务端发现,现在流行用 etcd 或 consul。 |
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
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.
做了以下改进:
net/http/pprof的BUG。io.Copy前就释放掉bufio.Reader,节约内存并提高对象从用性。加密包的引用路径是
github.com/idada/frontd/aces256cbc,直接合并可能会有问题,但目前不知道怎么处理。可能还是需要先合并再修改提交。