Skip to content

fix: plugins repeats the rewrite phase when consumer configs with a plugin#12048

Merged
Baoyuantop merged 5 commits intoapache:masterfrom
hachi029:master
Apr 3, 2025
Merged

fix: plugins repeats the rewrite phase when consumer configs with a plugin#12048
Baoyuantop merged 5 commits intoapache:masterfrom
hachi029:master

Conversation

@hachi029
Copy link
Contributor

@hachi029 hachi029 commented Mar 13, 2025

Description

plugins repeats the rewrite phase when consumer configs with a plugin.

Fixes # (issue)

#10945 #10873

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 Mar 13, 2025
@Baoyuantop Baoyuantop requested a review from membphis March 19, 2025 04:22
membphis
membphis previously approved these changes Mar 19, 2025
@hachi029 hachi029 requested a review from nic-6443 March 25, 2025 02:14
@Baoyuantop Baoyuantop changed the title fix: plugins repeats the rewrite phase when consumer configs with a p… fix: plugins repeats the rewrite phase when consumer configs with a plugin Apr 2, 2025
@@ -0,0 +1,104 @@
#
Copy link
Member

Choose a reason for hiding this comment

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

You can make a try, give this file a meaningful name

@Baoyuantop Baoyuantop merged commit 4d478c6 into apache:master Apr 3, 2025
32 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in ⚡️ Apache APISIX Roadmap Apr 3, 2025
laz-xyr pushed a commit to laz-xyr/apisix that referenced this pull request Jun 19, 2025
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:L This PR changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants