Skip to content

fix: use plugin.get to fetch plugin configured in multi-auth plugin#11794

Merged
moonming merged 2 commits into
apache:masterfrom
Revolyssup:revolyssup/custom-plugin-in-multi-auth
Dec 5, 2024
Merged

fix: use plugin.get to fetch plugin configured in multi-auth plugin#11794
moonming merged 2 commits into
apache:masterfrom
Revolyssup:revolyssup/custom-plugin-in-multi-auth

Conversation

@Revolyssup

@Revolyssup Revolyssup commented Nov 29, 2024

Copy link
Copy Markdown
Contributor

Description

use plugin.get to fetch plugin configured in multi-auth plugin

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)

@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. bug Something isn't working plugin labels Nov 29, 2024
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Nov 29, 2024

@membphis membphis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@moonming
moonming merged commit c11f602 into apache:master Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working plugin size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants