-
Notifications
You must be signed in to change notification settings - Fork 566
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
Include requested URI config in ListenerConfig; fix config keys #8371
Conversation
@@ -154,7 +159,7 @@ public Builder trustedProxies(AllowList trustedProxies) { | |||
* @param discoveryTypes discovery types to use | |||
* @return updated builder | |||
*/ | |||
@ConfiguredOption | |||
@ConfiguredOption(key = "types") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think a better option would be to have a new method called types
, and deprecate the discoveryTypes
one.
This way we align config key and method name, as should be the default in Helidon
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed.
…don-io#8371) * Include requested URI config in ListenerConfig; fix config keys * Forward-port requested URI discovery tests from 3.x * Review comments
…don-io#8371) * Include requested URI config in ListenerConfig; fix config keys * Forward-port requested URI discovery tests from 3.x * Review comments
Description
Resolves #8367
Resolves #8372
The
ListenerConfigBlueprint
declaredrequestedUriDiscoveryContext
but did not annotate it with@Option.Configured
so Helidon never processed configuration for it.Changes:
requestedUriDiscoveryContext
onListenerConfigBlueprint
so it is configured asrequested-uri-discovery
and optional.RequestedUriDiscoveryContext
. It does not yet use the new builder approach and needs to at some point; to get this PR completed more quickly that is not part of this PR.Documentation
Bug fix; no doc impact.