Skip to content

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

Merged
shreemaan-abhishek merged 6 commits into
apache:masterfrom
shreemaan-abhishek:feat/append-request_id
Jan 26, 2026
Merged

feat: support apisix_request_id variable with request-id plugin#12931
shreemaan-abhishek merged 6 commits into
apache:masterfrom
shreemaan-abhishek:feat/append-request_id

Conversation

@shreemaan-abhishek

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

Copy link
Copy Markdown
Contributor

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)

@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 <[email protected]>
@Revolyssup @shreemaan-abhishek
feat: add apisix_request_id variable with request-id plugin
b143f22 
Signed-off-by: Abhishek Choudhary <[email protected]>
@shreemaan-abhishek
log
24a9cad 
Signed-off-by: Abhishek Choudhary <[email protected]>
@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 <[email protected]>
@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
Comment thread apisix/cli/ngx_tpl.lua Outdated
@shreemaan-abhishek
request line
270f962 
Signed-off-by: Abhishek Choudhary <[email protected]>
@shreemaan-abhishek
shreemaan-abhishek merged commit b971b19 into apache:master Jan 26, 2026
34 of 36 checks passed
@Baoyuantop Baoyuantop mentioned this pull request Feb 11, 2026
Closed
5 tasks
wistefan pushed a commit to wistefan/apisix that referenced this pull request Jun 16, 2026
@shreemaan-abhishek @wistefan
feat: support apisix_request_id variable with request-id plugin (ap…
161f601 
…ache#12931)
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