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

Improvement server performance and usability #2

Merged
merged 8 commits into from Sep 7, 2017
Merged

Improvement server performance and usability #2

merged 8 commits into from Sep 7, 2017

Conversation

iperevozchikov
Copy link
Contributor

@iperevozchikov iperevozchikov commented Sep 7, 2017 •
edited

Changes:

  1. Using of a "bouncy" library was removed because it has bad performance: each request was executed over 3-5 seconds. Now it's around of 300-500ms.
  2. Working with input variables were improved with creating bin runner for each part and collecting passed arguments via optimist library.
  3. Supporting run the server on a subdomain. Ex. blabla.domain.com it's the main point and myawesome.blabla.domain.com it's tunneled resource.
  4. Adding sample for configuring Nginx endpoint.

@iperevozchikov
Updated deps versions
92f5ae9
@iperevozchikov
Add optimist dependency
c392736
@iperevozchikov
Refactor client to use options instead of hardcoded variables
f80c56a
@iperevozchikov
Many improvements:
f95f40b 
1. Refactor server to use options instead of hardcoded variables.
2. Remove using bouncy library, because it's have bad performance.
3. Instead using raw http server and streams.
4. Add support for run server on subdomain
@iperevozchikov
Add executables runners for client and server
ca46753
@iperevozchikov
Add sample config for nginx
0d6c091
@iperevozchikov
Change docker command
3b4e503
@iperevozchikov
Fix naming bugs
4e10c3b
@ericbarch
Copy link
Owner

Really appreciate all the work you did! I originally created this as a proof of concept, so this is great to get some optimizations in place 👍

@ericbarch ericbarch merged commit 28ae61d into ericbarch:master Sep 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@iperevozchikov @ericbarch