-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add TOML and JSON support for quarkus configuration #6
Comments
Thanks @radcortez . From my selfish point of view:
|
YAML is already covered, right? :) |
@radcortez Well currently working on it, and I do not seem to get it to work, only keys which are written to consul as To provide more context:
Keys written to K/V as: app:
subkey: value are not being picked up by quarkus. |
I think I have tracked down the issue as far as I understand it. @SuppressWarnings({ "rawtypes", "unchecked" })
private static Map<String, String> readProperties(String rawData) {
try (StringReader br = new StringReader(rawData)) {
final Properties properties = new Properties();
properties.load(br);
return (Map<String, String>) (Map) properties;
} catch (IOException e) {
throw new UncheckedIOException(e);
}
} when using |
@radcortez @hrstoyanov Please check |
@gjrtimmer I believe you wanted to add these comments to #46? This issue has nothing to do with either YAML or Consul. |
Originated by quarkusio/quarkus#14330
The text was updated successfully, but these errors were encountered: