Skip to content

Fortinet FortiGate - Official Support for Capirca with Local-In Policy Support - #364

Open
pc48m8n1 wants to merge 18 commits into
google:masterfrom
pc48m8n1:fortigate-localin
Open

Fortinet FortiGate - Official Support for Capirca with Local-In Policy Support#364
pc48m8n1 wants to merge 18 commits into
google:masterfrom
pc48m8n1:fortigate-localin

Conversation

@pc48m8n1

Copy link
Copy Markdown
Contributor

The Fortinet team is pleased to provide this contribution to Google Capirca. We welcome your guidance and suggestions to this contribution and are committed to the processes at hand.

We have only included three sample .POL files, per the observed standards and including the local-in policies for ipv4 and ipv6.

Thank you.

@loulecrivain

Copy link
Copy Markdown

@pc48m8n1 I'm working on another project (Nautobot Firewall App integration for FortiGate) and we would be interested in having this feature merged too!

I'm pinging you since it seems your feature branch is out of-sync and has conflicts (nothing too big, see diff below).

Hope someone at Google will take some time to look at your PR soon (do you know anyone to ping?).

diff --cc capirca/lib/policy.py
index 72a47a6,6a943aa..0000000
--- a/capirca/lib/policy.py
+++ b/capirca/lib/policy.py
@@@ -1798,12 -1737,8 +1819,17 @@@ class VarType
DECAPSULATE = 67
SOURCE_SERVICE_ACCOUNTS = 68
VERSA_APPLICATION = 69
++<<<<<<< HEAD
+ TRAFFIC_CLASS = 70
+ NEXT_HOP_GROUP = 71
+ NEXT_INTERFACE = 72
+ POLICE_KBPS = 73
+ POLICE_BURST = 74
+ POLICE_PPS = 75
++=======
+ APPLICATION_ID = 70
+ INTERFACE = 71
++>>>>>>> pc48m8n1/fortigate-localin

@abhindesabhindes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for the generator contribution, with the in-depth feature support being added.

Comment threadcapirca/lib/iptables.py
Comment threadcapirca/lib/policy.py Outdated
Comment threadcapirca/lib/policy.py Outdated
Comment threadcapirca/lib/fortigate.py
Comment threadcapirca/lib/fortigate.py Outdated
Comment threadtests/lib/fortigate_test.py
Comment threadtests/lib/fortigate_test.py
Comment threadtests/lib/fortigatelocalin_test.py Outdated
Comment threadcapirca/lib/fortigate.py
Comment threadcapirca/lib/fortigate.py
@pc48m8n1

Copy link
Copy Markdown
ContributorAuthor

Hi @abhindes, please review my latest commit for changes as per your request.

Comment threadcapirca/lib/iptables.py Outdated
Comment threadcapirca/lib/fortigate.py Outdated
Comment threadcapirca/lib/fortigate.py
@pc48m8n1

Copy link
Copy Markdown
ContributorAuthor

Hi @abhindes I have made the change to check the platform instead for local-in policy.

Comment threadcapirca/lib/policy.py Outdated
@pc48m8n1

Copy link
Copy Markdown
ContributorAuthor

@abhindes They have been removed. Please review.

Comment threadcapirca/lib/policy.py
@abhindesabhindes self-assigned this May 27, 2025
Comment threadcapirca/lib/fortigate.py Outdated
Comment threadcapirca/lib/fortigate.py
copybara-serviceBot pushed a commit that referenced this pull request Jun 2, 2025
Sign up for freeto 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

@pc48m8n1@loulecrivain@abhindes