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 AppService trait and use it to implement the HTTP service #59

Merged
merged 2 commits into from
May 4, 2024

Conversation

spencewenski
Copy link
Member

Add an AppService trait to allow consumers to define custom services that will be run as tokio tasks. This is similar (I believe) to Laraval's Provider concept.

Use the AppService trait to add an HTTP server using Axum instead of putting http/axum specific methods directly on the App trait

Add an `AppService` trait to allow consumers to define custom services
that will be run as tokio tasks. This is similar (I believe) to
Laraval's `Provider` concept.

Use the `AppService` trait to add an HTTP server using Axum instead of
putting http/axum specific methods directly on the `App` trait

- This will help with #9
  (and we may be able to resolve it)
- This will resolve #19
- Allow consumers to register services using the `ServiceRegistry`.
- Update the config structure to add a `service` config, under which the
  `http` service config belongs, along with future services
@spencewenski spencewenski enabled auto-merge (rebase) May 4, 2024 02:06
@spencewenski spencewenski merged commit 6d7c1ee into main May 4, 2024
3 checks passed
@spencewenski spencewenski deleted the services branch May 4, 2024 02:13
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