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 MuAPI Config::get Error #755

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

fix MuAPI Config::get Error #755

wants to merge 4 commits into from

Conversation

charleyzhu
Copy link

app/Controllers/Mu/UserController.php
if (Config::get('log_traffic_dynamodb')) { try { $client = new DynamoTrafficLog(); $id = $client->store($u, $d, $nodeId, $id, $totalTraffic, $rate); $res['id'] = $id; } catch (\Exception $e) { $res['msg'] = $e->getMessage(); Logger::error($e->getMessage()); } }
not implemented Config Get static function

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 8.171% when pulling 019a242 on charleyzhu:dev into d2977da on orvice:dev.

@charleyzhu
Copy link
Author

请问下 这个几个pull request是没通过吗?

@orvice
Copy link
Owner

orvice commented Jul 17, 2017

dockerfile没必要添加mysql判断, depends_on 会决定启动顺序了

另外docker-compose.yml里redis端口不应该映射出去

@charleyzhu
Copy link
Author

depends_on是决定了启动顺序,但是如果使用docker-compose启动应用,在ss-panel启动完成然后导入数据库的时候你会发现这时候sspanel-mysql是起来了,但是mysql的服务并没有启动完全,这时候并不能完成导入操作和创建管理员的操作

@charleyzhu charleyzhu closed this Jul 17, 2017
@charleyzhu charleyzhu reopened this Jul 17, 2017
@orvice
Copy link
Owner

orvice commented Jul 17, 2017

那加下restart也可以了,mysql链接失败,自动重启即可。

@charleyzhu
Copy link
Author

还有
app/Controllers/Mu/UserController.php
里面使用 if (Config::get('log_traffic_dynamodb'))
这个Config::get这个static方法并不存在

@charleyzhu
Copy link
Author

redis是我理解错误,我以为这个是给服务器用的,所以映射出去了,但是如果endpoint加入判断mysql服务是否启动成功可以提示docker-compose的体验

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 8.171% when pulling 019a242 on charleyzhu:dev into d2977da on orvice:dev.

@orvice
Copy link
Owner

orvice commented Jul 17, 2017

1 mu v1是有问题,v4开发的时候仅考虑到了mu v2

2 首先引入新依赖本来就不太好,而且 有depend on,mysql先启动就没有问题了,有restart重启机制,也能够跑起来,而且其实mysql没启动完成这种情况很少。

@charleyzhu
Copy link
Author

mysql的属性里面也没有指定sspanel这个库的用户和密码,连接的时候还会提示用户名和密码错误
不能用root用户来连接吧?在docker-compose中也指定了DB_USERNAME和DB_PASSWORD使用sspanel作为连接mysql的用户和密码

@orvice
Copy link
Owner

orvice commented Jul 17, 2017

使用root情况一般是,一个web服务一个mysql实例

如果是高级用户,完全可以自己定制docker-compose,自己创建msyql账户配置

@charleyzhu
Copy link
Author

"而且其实mysql没启动完成这种情况很少",偏偏我就遇到了 :( 使用的是vultr.com的vps,1G内存,1核CPU,可能配置太差了,docker-compose定制确实是可行的,可能我想多了,

@charleyzhu
Copy link
Author

app/Controllers/Mu/UserController.php
里面使用 if (Config::get('log_traffic_dynamodb'))
这个Config::get这个static方法并不存在
这个怎么处理呢?我是加了个方法

@charleyzhu
Copy link
Author

还有MuV2的获取用户的方法里面没有ret

@orvice
Copy link
Owner

orvice commented Jul 17, 2017

@charleyzhu mu config的问题我会改, mu v2里已经不需要retle

@charleyzhu
Copy link
Author

charleyzhu commented Jul 17, 2017

@orvice 现在V4版本使用ss-go的时候如果config返回不是json,ss服务器提示以后端口监听会莫名的取消,但是mu程序还是保持运行,所以我才加了config的get方法,这个问题还没找到在哪里,另外问下以后V2版本的api是不是依据http 状态码来区分是否请求成功?

@orvice
Copy link
Owner

orvice commented Jul 18, 2017

是 现在ss客户端都用状态码判断成功了

@charleyzhu
Copy link
Author

@orvice 好的,谢谢!自己在练习重复造轮子,重写ss-go-mu

@michaelauto
Copy link

O大神求解,v4.0.0 用hyperapp教程布置
问题一:用户分配端口正确,但是二维码获取的端口错误,例:用户端口分配为1026,二维码获取的是1027
问题二:默认邀请码数量设置不了,不管设置成什么每个人都还是可以获取10个邀请码,希望可以设置成0,关闭这个功能

@stone1949
Copy link

stone1949 commented Oct 31, 2017

求助:搭建完V4.0.0的前端后,访问http://IP/admin 后不能正常停在管理员界面,又返回带主页,日志显示GET /api/users/null/ HTTP/1.1" 401 44 ,这是什么原因呢?

其他界面都可以正常访问,admin账户也已经建好了。谢谢答复。

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.

5 participants