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

一系列功能性和安全性改动 #508

Closed
wants to merge 3 commits into from

Conversation

snowie2000
Copy link
Contributor

@snowie2000 snowie2000 commented Apr 15, 2020

去掉nps原有的http代理,改用httputil.reverseproxy,可以支持ws
去掉基于server.key和server.pem的tls连接,改为在启动时动态生成证书(代码部分来自gost)

新的反向代理支持了流量统计和用户认证等,但不支持连接数统计。不支持http_cache。

所有改动不需要改动原有客户端,和现有客户端完全兼容

replaced self-made http reverseproxy with a more robust and versatile one.
dynamically generate cert for client-server tls encryption
@snowie2000
Copy link
Contributor Author

本pr解决问题#507 #502

@ffdfgdfg ffdfgdfg changed the base branch from master to dev April 15, 2020 11:55
@ffdfgdfg
Copy link
Collaborator

tls可以,http的改动确实有点大,得再看看

@ffdfgdfg
Copy link
Collaborator

采纳tls修改,撤回http反代修改,主要有如下考虑

  • 目前版本主要是稳定性更新,功能更新不涉及到主体功能,做出一个相对稳定版本后着手进行重构,有很多代码遗留问题的,websocket的支持也是放到重构的计划里
  • 功能的缺失,不仅仅是少了cache,keepalive也少了,尝试补全功能失败了,功能一缺,又会有相关issue冒出来而且短期内没法解决

ffdfgdfg added a commit that referenced this pull request Apr 30, 2020
@snowie2000 snowie2000 closed this May 1, 2020
kiririx pushed a commit to kiririx/nps that referenced this pull request Jul 26, 2022
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.

None yet

2 participants