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

Feat: Add CIDR related actions #849

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

L-Mario564
Copy link

Addresses #342.

This PR adds 3 actions:

  • ipCidr
  • ipv4Cidr
  • ipv6Cidr

These actions are for validating that an IP address is in CIDR notation.

The regular expressions added were verified with https://makenowjust-labs.github.io/recheck/playground/ and https://devina.io/redos-checker and both mark them as safe.

Some tests were added and the actions themselves have been added to the docs website.

@fabian-hiller fabian-hiller self-assigned this Sep 26, 2024
@fabian-hiller fabian-hiller added the enhancement New feature or request label Sep 26, 2024
@fabian-hiller
Copy link
Owner

Thank you for this contribution! I will try to review and merge it next week.

@fabian-hiller fabian-hiller force-pushed the main branch 2 times, most recently from 7a97dd0 to b5f7f4d Compare October 3, 2024 17:45
@fabian-hiller
Copy link
Owner

I did not forget this PR. Sorry for the delay. I will try to review it soon.

@fabian-hiller
Copy link
Owner

I just want to let you know that I am focusing on our v1 release first before reviewing this PR.

@L-Mario564
Copy link
Author

I just want to let you know that I am focusing on our v1 release first before reviewing this PR.

Sounds good. Thanks for the updates. 🙂

@fabian-hiller fabian-hiller force-pushed the main branch 2 times, most recently from f41426c to d713dfe Compare January 4, 2025 00:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants