Skip to content

Conversation

@pottekkat
Copy link
Member

Signed-off-by: Navendu Pottekkat navendupottekkat@gmail.com

Description

Updates the documentation of the "ip-restriction" Plugin.

Child PR of #6734

Signed-off-by: Navendu Pottekkat <navendupottekkat@gmail.com>
Signed-off-by: Navendu Pottekkat <navendupottekkat@gmail.com>
Copy link
Member

@spacewander spacewander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it necessary to submit a PR per plugin, while the changes are similar?

@pottekkat
Copy link
Member Author

See discussion in #6877 (comment)

Copy link
Member

@juzhiyuan juzhiyuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

- Plugin
- IP restriction
- ip-restriction
description: This document contains information about the Apache APISIX ip-restriction Plugin.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could even improve this section with some semantic keywords here

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. I'm currently keeping it similar. But after we have done some research on what works best, we can definitely change them.

@spacewander spacewander merged commit 699e4c5 into apache:master Apr 25, 2022
@pottekkat pottekkat deleted the docs/plugin-security-ip-restriction/6734 branch April 25, 2022 11:53
Liu-Junlin pushed a commit to Liu-Junlin/apisix that referenced this pull request May 20, 2022
Signed-off-by: Navendu Pottekkat <navendupottekkat@gmail.com>
@guitu168 guitu168 mentioned this pull request Jun 6, 2022
5 tasks
spacewander pushed a commit that referenced this pull request Jun 30, 2022
Signed-off-by: Navendu Pottekkat <navendupottekkat@gmail.com>
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.

3 participants