Skip to content

Commit

Permalink
fix(panos.policies.SecurityRule): Param hip_profiles is actually re…
Browse files Browse the repository at this point in the history
…moved in PAN-OS 10.0
  • Loading branch information
shinmog committed May 12, 2022
1 parent ef8c9fc commit 31eb2a6
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions panos/policies.py
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,7 @@ class SecurityRule(VersionedPanObject):
tozone (list): To zones
source (list): Source addresses
source_user (list): Source users and groups
hip_profiles (list): (PAN-OS 10.1.4-) GlobalProtect host integrity profiles
hip_profiles (list): (PAN-OS 10.0.0-) GlobalProtect host integrity profiles
destination (list): Destination addresses
application (list): Applications
service (list): Destination services (ports) (Default:
Expand Down Expand Up @@ -388,12 +388,12 @@ def _setup(self):
)
)

# 10.1.5 drops support for hip-profiles,
# 10.0.0 drops support for hip-profiles,
# so we want to make sure we don't include it in the request
# body that we send to the api
for param in params:
if param.name == "hip_profiles":
param.add_profile("10.1.5", exclude=True)
param.add_profile("10.0.0", exclude=True)
break

params.append(
Expand Down

0 comments on commit 31eb2a6

Please sign in to comment.