Skip to content

fix: load full data during init_worker phase require a new apisix-runtime#12678

Merged
nic-6443 merged 3 commits into
apache:masterfrom
nic-6443:nic/upgrade-nginx-mogule
Oct 16, 2025
Merged

fix: load full data during init_worker phase require a new apisix-runtime#12678
nic-6443 merged 3 commits into
apache:masterfrom
nic-6443:nic/upgrade-nginx-mogule

Conversation

@nic-6443
Copy link
Copy Markdown
Member

@nic-6443 nic-6443 commented Oct 16, 2025

Description

I forgot to upgrade apisix-runtime when made this PR: #12523
image
this patch api7/apisix-nginx-module#108 is required to let this fix working finally. sorry for the previous test cases couldn't verify the real situation. I have optimized them to ensure that the fix is indeed effective.

Which issue(s) this PR fixes:

Fixes #

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:M This PR changes 30-99 lines, ignoring generated files. bug Something isn't working labels Oct 16, 2025
f
Signed-off-by: Nic <qianyong@api7.ai>
f
Signed-off-by: Nic <qianyong@api7.ai>
Comment thread t/cli/test_ci_only.sh
mkdir /root/apisix

cp -r ./* /root/apisix
cp -R ./* /root/apisix
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

for lint fix:
image

@nic-6443 nic-6443 merged commit 1af6ec4 into apache:master Oct 16, 2025
47 of 50 checks passed
@nic-6443 nic-6443 deleted the nic/upgrade-nginx-mogule branch October 16, 2025 09:28
jizhuozhi pushed a commit to jizhuozhi/apisix that referenced this pull request Oct 18, 2025
membphis added a commit that referenced this pull request May 12, 2026
The `ci/check_changelog_prs.ts` script captures only the first `#NNNN`
match per line. Four entries from the previous commit listed a related
PR (or an external-repo reference like `api7/apisix-nginx-module#108`)
before the entry's own PR number, so the check reported the entries'
own PRs as missing from CHANGELOG.md. Reorder so that the entry's
canonical PR appears first on the line:

- 3.15.0 #12678 (apisix-runtime init_worker)
- 3.13.0 #12244 (server-info deprecation)
- 3.10.0 #11095 (encrypted plugin fields expansion)
- 3.8.1  #11174 (forward-auth POST headers)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants