Skip to content

fix: plugin metadata add id value for etcd checker#11452

Merged
moonming merged 1 commit into
apache:masterfrom
ronething:fix/plugin_metadata_admin
Aug 1, 2024
Merged

fix: plugin metadata add id value for etcd checker#11452
moonming merged 1 commit into
apache:masterfrom
ronething:fix/plugin_metadata_admin

Conversation

@ronething

@ronething ronething commented Jul 31, 2024

Copy link
Copy Markdown
Member

Description

Fixes #11216

fix part 1 of this issue by adding an id field to the plugin metadata with the value of the plugin name

This PR is a partial fork of #11430.

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

Comment thread apisix/plugins/sls-logger.lua
@moonming
moonming merged commit d0af27b into apache:master Aug 1, 2024
@ronething
ronething deleted the fix/plugin_metadata_admin branch August 1, 2024 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: the plugin metadata obtained from etcd has not been validated by schema

4 participants