Skip to content

129 add community requested source and destination objects #236

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

Conversation

alexis-UE
Copy link

@alexis-UE alexis-UE commented Mar 10, 2025

@rvveber @cruse1977

Pull Request

Related Pull Request

Rebase of #207 from latest dev branch (commit 404ad38)

Related Issue

closes #129

New Behavior

This Merge Request, enables to specify various new Source Entities for the Standard and Extended ACL Rules. It additionally enables specify the same Entities as a Destination for the Extended ACL Rule. The new selectable entities are all from the IPAM model group:

  • IP-Range
  • IP-Address
  • Aggregate
  • Service

I have built in constraints and validation to prevent assigning more than one source/destination. Therefore adding community requested tities, while keeping the functionality the same. No changes to the API, except these newly added fields: source_iprange, ource_ipaddress, source_aggregate, source_serviceanddestination_iprange, destination_ipaddress, destination_aggregate, estination_service ...

Contrast to Current Behavior

Previously you could only assign a Prefix as Source and Destination. Now you can assign a Prefix, an IP-Range, an IP-Address, an gregate or a Service. ...

Discussion: Benefits and Drawbacks

...

Changes to the Documentation

...

Proposed Release Note Entry

Added requested models for selection as source/destination: IP-Range, IP-Address, Aggregate, Service. ...

Double Check

  • I have explained my PR according to the information in the comments
    or in a linked issue.
  • My PR targets the dev branch.

rvveber added 30 commits March 10, 2025 16:46
…urces; Add validation to restrict to a single Source
…urces; Add validation to restrict to a single Source
…stinations; Add validation to restrict to a single Destination
… Destination column that will display any Destination
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🎉 Thanks for opening this pull request! We really appreciate contributors like you! 🙌

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.

[Feature]: source and destination range for acl have to be a prefix in Netbox?
2 participants