Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add IPv4 whitelist/blacklist plugin ip-restriction #398

Merged
merged 5 commits into from
Aug 19, 2019
Merged

Add IPv4 whitelist/blacklist plugin ip-restriction #398

merged 5 commits into from
Aug 19, 2019

Conversation

chnliyong
Copy link
Contributor

@chnliyong chnliyong commented Aug 12, 2019

Add IP whitelist/blacklist plugin ip-restriction, the code mainly migrate from kong's ip-restriction.

Use lua-resty-iputils check CIDR v4 format as the json-schema cannot check CIDR format.

Now IPv6 doesn't support.

@membphis
Copy link
Member

@moonming do you agree we can merge this PR first, and fix those small things with new PR.

@moonming
Copy link
Member

@moonming do you agree we can merge this PR first, and fix those small things with new PR.

I will modify this PR directly.

@moonming moonming merged commit f26f8ba into apache:master Aug 19, 2019
@chnliyong
Copy link
Contributor Author

@moonming I have discussed with @membphis, and plan to use another implementation, and add IPv6 support,so I may PR the same plugin soon later.

@moonming
Copy link
Member

@chnliyong ok, already merged this PR first.

nic-chen pushed a commit to nic-chen/apisix that referenced this pull request Aug 22, 2019
@moonming moonming added this to the 0.7 milestone Aug 27, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants