-
Couldn't load subscription status.
- Fork 2.7k
fix(plugin_metadata): Ensure enable_data_encryption initialization + Querying issue #12624
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
base: master
Are you sure you want to change the base?
Conversation
|
@Baoyuantop fix ci error,when merging the upstream, due to this pr #12603, no longer populates default values when writing, rename node.name to id to avoid decryption failure |
|
@Baoyuantop CI Tars Need I fix it? |
This seems unrelated to this PR. I'll start reviewing this PR when I have free time. |
Description
plugin.enable_data_encryption does not initialization when no plugin configuration for routes, plugin_metadata, etc,When not initialized, querying non-existent plugin_metadata via the admin_api will report http 500
Just like encrypt_conf and decrypt_conf, Instead of directly accessing plugin.enable_data_encryption, call the method enable_gde(),decrypt_params metadata is changed to the same code style as node and list to avoid nil
Which issue(s) this PR fixes:
Fixes #12618
Checklist