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

GatewayClient #18

Closed
sm2017 opened this issue Dec 5, 2016 · 3 comments
Closed

GatewayClient #18

sm2017 opened this issue Dec 5, 2016 · 3 comments

Comments

@sm2017
Copy link

sm2017 commented Dec 5, 2016

Which is newer?
https://github.com/walkor/GatewayClient/blob/master/Gateway.php
OR
https://github.com/walkor/GatewayWorker/blob/master/src/Lib/Gateway.php

And I think it is better to add GatewayClient as a dependency of GatewayWorker using composer

@walkor
Copy link
Owner

walkor commented Dec 5, 2016

They are not exactly the same library.
GatewayClient/blob/master/Gateway.php is SDK for any project run on apache or php-fpm to control GatewayWorker(send message, get session, set session, join group etc).
GatewayWorker/blob/master/src/Lib/Gateway.php is API witch must run on GatewayWorker.

All of them will be keep the synchronization update.

@sm2017
Copy link
Author

sm2017 commented Dec 5, 2016

In https://github.com/walkor/GatewayClient we have

Gateway::$registerAddress = 'x.x.x.x:xx';

Is it required to set registerAddress for GatewayWorker/blob/master/src/Lib/Gateway.php?
Now I don't set it but it works for me in Distributed Deployment , I use GatewayWorker/blob/master/src/Lib/Gateway.php form business workers to send message to clients

What happens if I set it?

@walkor
Copy link
Owner

walkor commented Dec 5, 2016

Is it required to set registerAddress for GatewayWorker/blob/master/src/Lib/Gateway.php?

Gateway::$registerAddress will be automatically obtained from $businessworker->registerAddress witch defined in start_businessworker.php.

@walkor walkor closed this as completed Apr 24, 2017
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

No branches or pull requests

2 participants