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

Fix issue https://github.com/XX-net/XX-Net/issues/1422 #1433

Merged
merged 2 commits into from
Nov 22, 2015
Merged

Fix issue https://github.com/XX-net/XX-Net/issues/1422 #1433

merged 2 commits into from
Nov 22, 2015

Conversation

Cat7373
Copy link
Contributor

@Cat7373 Cat7373 commented Nov 21, 2015

成功实现GAE部分的重启,后台还没有。

@@ -182,23 +182,39 @@ def do_CONNECT(self):

class HTTPServer():
def __init__(self, address, handler, args=()):
self.first = True
Copy link
Contributor

Choose a reason for hiding this comment

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

此处可以改为 self.socket = None
后面直接拿socket是否为None 来判断

@xxnet
Copy link
Contributor

xxnet commented Nov 21, 2015

把后台也搞定了,我再合并

@Cat7373
Copy link
Contributor Author

Cat7373 commented Nov 21, 2015

没想到什么不修改python内的lib的方法...
这货捕获了关键的错误然后直接return了.........

@Cat7373
Copy link
Contributor Author

Cat7373 commented Nov 21, 2015

在研究能不能通过覆盖方法来实现- -
有什么办法能覆盖init呢- -

@Cat7373
Copy link
Contributor Author

Cat7373 commented Nov 21, 2015

好像是我想多了,好像不需要覆盖。。

@Cat7373
Copy link
Contributor Author

Cat7373 commented Nov 21, 2015

@xxnet 基本做好了- -

@xxnet xxnet merged commit 725a699 into XX-net:master Nov 22, 2015
@xxnet
Copy link
Contributor

xxnet commented Nov 22, 2015

辛苦了,已经合并。

这个目录结构还要再调整,自己写的几个基础库,是应该放在一个公共地方,
比如xlog, simple_http_server,这样launcher, gae_proxy都可以直接用了。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants