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

Update the subsystem to be exposing CDI beans using gRPC #299

Open
darranl opened this issue Oct 18, 2024 · 2 comments
Open

Update the subsystem to be exposing CDI beans using gRPC #299

darranl opened this issue Oct 18, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@darranl
Copy link
Contributor

darranl commented Oct 18, 2024

In theory it should be possible to register a CDI Extension to process CDI beans as they are defined, with this we can check for annotations / interfaces implemented etc.. and handle the gRPC registration.

This issue is to cover the current deployment type, follow up issues may be created for:

  • Injection of gRPC resources if any are useful.
  • Alternate deployment approaches / annotations.
@darranl darranl added the enhancement New feature or request label Oct 18, 2024
@darranl darranl added this to the 0.1.5 milestone Oct 18, 2024
@darranl darranl self-assigned this Oct 18, 2024
@jamezp
Copy link
Contributor

jamezp commented Oct 18, 2024

We probably want to handle the gRPC registration via a DUP and Jandex. While I still think a CDI Extension could be useful, I'm not sure we want to determine registration based on that.

@darranl
Copy link
Contributor Author

darranl commented Oct 22, 2024

+1 the decision to activate gRPC for the deployment will need to be a traditional DUP / Jandex scan - we only want to activate the CDI extension after we have confirmed the deployment needs gRPC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants