-
Notifications
You must be signed in to change notification settings - Fork 38.5k
Make HttpServiceGroupAdapter implementations @Reflective #34842
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
base: main
Are you sure you want to change the base?
Make HttpServiceGroupAdapter implementations @Reflective #34842
Conversation
Signed-off-by: Olga Maciaszek-Sharma <[email protected]>
063439c
to
9b3dcb3
Compare
Scheduled for 7.0 M6 as a confirmed issue, but need to confirm the preferred way to mark these classes with some help from @snicoll. |
I don't like it because the link between the use of the reflection and the annotation is not clear. It only works because the two types are eventually exposed as beans I suppose? To me that's an implementation detail and whoever is doing the reflection should register the hints. As discussed with Rossen, I'll share an alternative fix later today. |
@OlgaMaciaszek how do I reproduce? The server starts fine in a native image, the client doesn't compile |
FW has had changes in between and also Boot snapshots have been released. I have forked this and added updates: https://github.com/OlgaMaciaszek/testjars-demo/tree/import-http-services-upgraded. Use the However, there have been quite some changes in FW between this PR being created and now. As a result, when I try to reproduce now, I no longer get:
which seems to be a new issue. |
Fixes startup issue while running Interface Client apps with native images.
Reproducer: https://github.com/scratches/testjars-demo/tree/import-http-services