Skip to content

feat: support apisix_request_id variable with request-id plugin #12931

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.

Sign up for GitHub

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

Jump to bottom
Merged
shreemaan-abhishek merged 6 commits into from
Jan 26, 2026
Merged

feat: support apisix_request_id variable with request-id plugin #12931

shreemaan-abhishek merged 6 commits into from
Jan 26, 2026
+105 −11

Conversation

@shreemaan-abhishek
Copy link
Contributor

@shreemaan-abhishek shreemaan-abhishek commented Jan 22, 2026 •
edited
Loading

Description

Record request_id in the error log to facilitate user troubleshooting.

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)

@Revolyssup @shreemaan-abhishek
fix: append apisix_request_id on access_log_format
f17bf36 
Signed-off-by: Abhishek Choudhary <shreemaan.abhishek@gmail.com>
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. bug Something isn't working labels Jan 22, 2026
shreemaan-abhishek and others added 3 commits January 22, 2026 12:23
@shreemaan-abhishek
grep e
35e2bdc 
Signed-off-by: Abhishek Choudhary <shreemaan.abhishek@gmail.com>
@Revolyssup @shreemaan-abhishek
feat: add apisix_request_id variable with request-id plugin
b143f22 
Signed-off-by: Abhishek Choudhary <shreemaan.abhishek@gmail.com>
@shreemaan-abhishek
log
24a9cad 
Signed-off-by: Abhishek Choudhary <shreemaan.abhishek@gmail.com>
@shreemaan-abhishek shreemaan-abhishek changed the title fix: append apisix_request_id on access_log_format feat: support apisix_request_id variable with request-id plugin Jan 22, 2026
@shreemaan-abhishek
comma
f28850a 
Signed-off-by: Abhishek Choudhary <shreemaan.abhishek@gmail.com>
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Jan 22, 2026
nic-6443
nic-6443 reviewed Jan 22, 2026
View reviewed changes
@shreemaan-abhishek
request line
270f962 
Signed-off-by: Abhishek Choudhary <shreemaan.abhishek@gmail.com>
@shreemaan-abhishek shreemaan-abhishek merged commit b971b19 into apache:master Jan 26, 2026
34 of 36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nic-6443 nic-6443 nic-6443 approved these changes

@Baoyuantop Baoyuantop Baoyuantop approved these changes

@Revolyssup Revolyssup Revolyssup approved these changes

@AlinsRan AlinsRan Awaiting requested review from AlinsRan

Assignees

No one assigned

Labels

bug Something isn't working size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@shreemaan-abhishek @nic-6443 @Baoyuantop @Revolyssup