Skip to content
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.

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

fix: update chaitin-waf.lua to support unix sock in host pattern #10161

Merged

Conversation

zclaiqcc
Copy link
Contributor

@zclaiqcc zclaiqcc commented Sep 5, 2023

Description

support UNIX sock URL like unix:/path/to/safeline/resources/detector/snserver.sock

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)

moonming
moonming previously approved these changes Sep 6, 2023
leslie-tsang
leslie-tsang previously approved these changes Sep 6, 2023
@zclaiqcc zclaiqcc dismissed stale reviews from leslie-tsang and moonming via 80ad672 September 6, 2023 06:27
@zclaiqcc zclaiqcc force-pushed the zclaiqcc-patch-chaitin-waf-unix-sock branch from 80ad672 to c86c776 Compare September 6, 2023 06:50
@zclaiqcc zclaiqcc force-pushed the zclaiqcc-patch-chaitin-waf-unix-sock branch from c86c776 to 9313864 Compare September 6, 2023 09:07
@monkeyDluffy6017 monkeyDluffy6017 merged commit 4da51ca into apache:master Sep 7, 2023
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants