Skip to content

feat(real-ip): support search recursive#6988

Merged
spacewander merged 19 commits into
apache:masterfrom
tangzhenhuang:real-ip-recursive
May 13, 2022
Merged

feat(real-ip): support search recursive#6988
spacewander merged 19 commits into
apache:masterfrom
tangzhenhuang:real-ip-recursive

Conversation

@tangzhenhuang

@tangzhenhuang tangzhenhuang commented May 6, 2022

Copy link
Copy Markdown
Contributor

Description

Support real ip recusive, see http://nginx.org/en/docs/http/ngx_http_realip_module.html

Fixes #6602

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)

@tzssangglass tzssangglass changed the title Real ip recursive feat(real-ip): support search recursive May 6, 2022
Comment thread apisix/plugins/real-ip.lua Outdated
Comment thread docs/en/latest/plugins/real-ip.md Outdated
Comment thread apisix/plugins/real-ip.lua Outdated
tangzhenhuang and others added 2 commits May 7, 2022 11:28
Thanks for review, it does reduce one loop.

Co-authored-by: 罗泽轩 <[email protected]>
Comment thread apisix/plugins/real-ip.lua Outdated
@tzssangglass

Copy link
Copy Markdown
Member

LGTM

spacewander
spacewander previously approved these changes May 8, 2022
@spacewander
spacewander requested a review from tzssangglass May 9, 2022 07:20
@spacewander spacewander added the wait for update wait for the author's response in this issue/PR label May 11, 2022
@tangzhenhuang
tangzhenhuang requested a review from spacewander May 11, 2022 06:37
Comment thread apisix/plugins/real-ip.lua Outdated
Comment thread apisix/plugins/real-ip.lua Outdated
Comment thread apisix/plugins/real-ip.lua
tzssangglass
tzssangglass previously approved these changes May 12, 2022
Comment thread apisix/plugins/real-ip.lua
Comment thread apisix/plugins/real-ip.lua Outdated
Comment thread apisix/plugins/real-ip.lua Outdated
@spacewander
spacewander merged commit 0842b69 into apache:master May 13, 2022
Liu-Junlin pushed a commit to Liu-Junlin/apisix that referenced this pull request May 20, 2022
Co-authored-by: tangzhenhuang <[email protected]>
Co-authored-by: 罗泽轩 <[email protected]>
spacewander added a commit that referenced this pull request Jun 30, 2022
Co-authored-by: tangzhenhuang <[email protected]>
Co-authored-by: 罗泽轩 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wait for update wait for the author's response in this issue/PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

request help: real_ip_recursive for real-ip plugin

4 participants