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.

Sign up for GitHub

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

Jump to bottom

Create more packages to allow users to depend only on what they need #18

Closed
juliendelplanque opened this issue Nov 15, 2019 · 2 comments · Fixed by #19
Closed

Create more packages to allow users to depend only on what they need #18

juliendelplanque opened this issue Nov 15, 2019 · 2 comments · Fixed by #19
Assignees
@gcotelli
Milestone
v3.0.0

Comments

@juliendelplanque
Copy link
Owner

I think packages are missing for the server:

  • JRPC-Server-TCP
  • JRPC-Server-HTTP
  • JRPC-Server-Core This package could be used if users only use the message-processor.

I would like to add support for web socket as transport layer in the future.

For the clients we will have similar packages:

  • JRPC-Client-TCP
  • JRPC-Client-HTTP

What do you think @gcotelli ?
@juliendelplanque juliendelplanque added this to the v3.0.0 milestone Nov 15, 2019
@gcotelli
Copy link
Collaborator

I think it's a good idea. I will do it and also add some groups to the baseline. Can you add me as an external collaborator to the project so I can assign me the issues I will be working on?

@juliendelplanque
Copy link
Owner Author

Invitation sent.

gcotelli added a commit to Mercap/JRPC that referenced this issue Nov 15, 2019
@gcotelli
Split JRPC-Server package into: JRPC-Server-Core , JRPC-Server-HTTP a…
6a64f74 
…nd JRPC-Server-TCP to the users can choose which transport use or just use the message processing support.

Fixes juliendelplanque#18
Fixed comment in JRPServer removing wording about inexistant methods
Fixes juliendelplanque#15
@gcotelli gcotelli mentioned this issue Nov 15, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gcotelli gcotelli

Labels
None yet
Projects
None yet
Milestone
v3.0.0
Development

Successfully merging a pull request may close this issue.

Package splitting and documentation updates Mercap/JRPC
2 participants
@gcotelli @juliendelplanque