Add legacy config root #213
Conversation
fjtirado
force-pushed
the
root_2_mapping
branch
from
f371fc0 to
8ff9c31
Compare
fjtirado
force-pushed
the
root_2_mapping
branch
from
db20622 to
ab3dc15
Compare
fjtirado
changed the title
There was a problem hiding this comment.
@fjtirado, should we switch to ConfigMapping as described here https://github.com/quarkusio/quarkus/wiki/Migration-Guide-3.14#for-extension-developers? Are there any problems on your side or was is just the quick&easy solution to enable the legacy-property?
There was a problem hiding this comment.
After some reading, i understand, that 'per default' all config-properties have to start with 'quarkus' - that might be breaking change.
There is a workaround https://github.com/orgs/quarkiverse/discussions/228#discussioncomment-10455441 implementing a smallrye config customizer...
But afaik our configs are all prefixed with 'quarkus' - so no problem for us
There was a problem hiding this comment.
@ChMThiel To be honest, I took the simplest approach, which was to specify the legacy property. Let me know if you try to change to ConfigMapping
No description provided.