Skip to content

fix(chaitin-waf): fix event_id is nil#11651

Merged
moonming merged 2 commits into
apache:masterfrom
xbingW:master
Feb 25, 2025
Merged

fix(chaitin-waf): fix event_id is nil#11651
moonming merged 2 commits into
apache:masterfrom
xbingW:master

Conversation

@xbingW

@xbingW xbingW commented Oct 15, 2024

Copy link
Copy Markdown
Contributor

Description

Fixes lua entry thread aborted when event_id is nil
image

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:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working plugin labels Oct 15, 2024
@xbingW

xbingW commented Oct 22, 2024

Copy link
Copy Markdown
Contributor Author

@bzp2010 @shreemaan-abhishek please review, thx

@xbingW

xbingW commented Nov 11, 2024

Copy link
Copy Markdown
Contributor Author

@bzp2010 @shreemaan-abhishek please review, thx

@xbingW

xbingW commented Jan 2, 2025

Copy link
Copy Markdown
Contributor Author

@membphis pls review it

@xbingW

xbingW commented Feb 10, 2025

Copy link
Copy Markdown
Contributor Author

can this PR be merged?

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