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

Add multi connection support #5

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

Conversation

smeinecke
Copy link
Contributor

  • add connection pooling for each registry.

- add connection pooling for each registry.
@smeinecke
Copy link
Contributor Author

Hmm, have to extend this a little more since it's unstable and does not handle new connections / login correctly - will update this once more.

return login status after first run of worker.
fix some bugs in pool handling + shutdown.
@smeinecke
Copy link
Contributor Author

Problems should be fixed now. Need to write some tests ;-)

@heytrav
Copy link
Owner

heytrav commented May 19, 2016

Cool, I'll merge it in once you've added some tests.

@heytrav
Copy link
Owner

heytrav commented Oct 16, 2016

Hey @smeinecke did you have anything you wanted to add to this?

@chrisschaub
Copy link
Collaborator

I might need to get this working with latest master unless you guys have a better approach?

@heytrav
Copy link
Owner

heytrav commented Mar 7, 2017

@chrisschaub I'm fine if you want to get this going in node-epp-server.js. I've moved to using node-epp-restful.js and rabbit-epp.js more since I'm running this in Docker Swarm.

@chrisschaub
Copy link
Collaborator

chrisschaub commented Mar 7, 2017 via email

@heytrav
Copy link
Owner

heytrav commented Mar 7, 2017

@chrisschaub I'm using swarm to scale the number of running instances that connect to the registry. I'm mainly using the rabbitmq version at the moment.

This is kind of an experiment for a side project at work.

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.

3 participants