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

#129 add community requested source and destination objects #207

Open
wants to merge 41 commits into
base: dev
Choose a base branch
from

Conversation

rvveber
Copy link

@rvveber rvveber commented Sep 2, 2024

Pull Request

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 to 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 entities, while keeping the functionality the same.
No changes to the API, except these newly added fields:
source_iprange, source_ipaddress, source_aggregate, source_service
and
destination_iprange, destination_ipaddress, destination_aggregate, destination_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 Aggregate 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.

… only one Destination Field can be specified
…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! 🙌

@rvveber rvveber force-pushed the 129_add_community_requested_source_and_destination_objects branch from 12dc111 to 7010069 Compare September 10, 2024 09:31
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?
1 participant