Skip to content

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

Merged
juststillthinking merged 2 commits into
apache:masterfrom
zclaiqcc:zclaiqcc-patch-chaitin-waf-unix-sock
Sep 7, 2023
Merged

fix: update chaitin-waf.lua to support unix sock in host pattern#10161
juststillthinking merged 2 commits into
apache:masterfrom
zclaiqcc:zclaiqcc-patch-chaitin-waf-unix-sock

Conversation

@zclaiqcc

@zclaiqcc zclaiqcc commented Sep 5, 2023

Copy link
Copy Markdown
Contributor

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
@juststillthinking
juststillthinking merged commit 4da51ca into apache:master Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants