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

Server consolidation #48

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

eccles
Copy link
Contributor

@eccles eccles commented Feb 16, 2024

Stuttering removed - GRPCServer renamed to just Server and similarly for http and restproxyserver.

grpcserver now accepts 'port' as an argument instead of getting the value from the environment.

AB#9031

@eccles eccles force-pushed the dev/paul/9031-server-function-outlining branch from f6256e9 to c62d684 Compare February 16, 2024 15:11
@eccles eccles force-pushed the dev/paul/9031-server-function-outlining branch from c62d684 to 788a3cf Compare March 25, 2024 10:27
@eccles eccles force-pushed the dev/paul/9031-server-function-outlining branch 2 times, most recently from 44dbaa4 to b1acef2 Compare April 10, 2024 10:39
Stuttering removed - GRPCServer renamed to just Server and
similarly for http and restproxyserver.

grpcserver now accepts 'port' as an argument instead of getting
the value from the environment.

metricsServer creation is now a single method

AB#9031
@eccles eccles force-pushed the dev/paul/9031-server-function-outlining branch from b1acef2 to d2c9851 Compare April 10, 2024 13:36
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.

1 participant