-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Wrong work is parsing configuration VaultRuntimeConfig #11921
Comments
CC @vsevel |
the plan is to move away from a dedicated parser. some support is expected fro the smallrye config project, as discussed here #9991 (comment) |
We were having a few issues with native mode that required a few changes. I think most of the work is done, so hopefully by the end of this week or next week. |
@radcortez @vsevel |
Class VaultConfigSource used it own parser for read app configuration with help regexp.
public static final Pattern SECRET_CONFIG_KV_PATH_PATTERN = Pattern .compile("^quarkus\\.vault\\.secret-config-kv-path\\.([^.]+)$");
VaultRuntimeConfig
@ConfigItem(name = "secret-config-kv-path.\"prefix\"") public Map<String, List<String>> secretConfigKvPrefixPath;
It regex does not parsing prefix string with dot char, example wrong work:
quarkus.vault.secret-config-kv-path."mp.jwt.verify"=myapps/app1
I think, need fix this regex or rewrite all class in microprofile style, similarly LogConfig class.
@ConfigItem(name = "category") @ConfigDocSection public Map<String, CategoryConfig> categories;
What do you think about this?
The text was updated successfully, but these errors were encountered: