Quarkus application set build time config by default #22071
-
|
Hi All, Custom configuration using @ConfigProperty could be updated in Config Editor at DEV UI using the previous quarkus version(e.g., 2.2) but Quarkus 2.5.1 won't let me do it since the variable is a read-only config in "Build time config" now.
Sample code: I was wondering if this is a new feature in 2.5.1 or a regression bug. |
Beta Was this translation helpful? Give feedback.
Replies: 2 comments 2 replies
-
|
This commit is what changed it: dc3171f Because of this code here: This is already fixed upstream, as I now ignore the 'editable' field, however that may have been a mistake, as you can't actually edit most config sources with an ordinal > 250. @phillip-kruger Maybe we should restore the UI checking of the 'editable' field that I removed, and instead maybe hard code a check for the 'Build time config' source? |
Beta Was this translation helpful? Give feedback.
-
|
Sounds good to me. |
Beta Was this translation helpful? Give feedback.
-
|
This is fixed in |
Beta Was this translation helpful? Give feedback.
This commit is what changed it: dc3171f
Because of this code here:
quarkus/extensions/vertx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/devmode/ConfigSourceName.java
Line 103 in d7a2f2b
This is already fixed upstream, as I now ignore the 'editable' field, however that may have been a mistake, as you can't actually edit most config sources with an ordinal > 250.
@phillip-kruger Maybe we should restore the UI checking of the 'editable' field that I removed, and instead maybe hard code a check for the 'Build time config' source?