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

Updates to management of services #61

Merged
merged 5 commits into from
May 8, 2024
Merged

Updates to management of services #61

merged 5 commits into from
May 8, 2024

Conversation

mamullen13316
Copy link
Collaborator

  • The create_service() method was updated to take a list of dicts containing port and protocol. This is because a single service can contain multiple port/protocols.

Example:

fw.create_service(name="FUNC_TESTSVC1", service_list=[{"dst_port": "1234", "protocol": "TCP"}, {"dst_port": "2222", "protocol": "TCP}])
  • The create_service() method was also updated to optionally support specification of source port(s). The default is 1:65535.
  • An update_service() method was created to allow adding or removing port/protocol pairs from the list, or they can be replaced.
fw.update_service(name="TEST_SVC", action="add", service_list=[{"dst_port": "2222", "protocol": "TCP"}])
  • Updated create_service() tests

@mamullen13316 mamullen13316 merged commit c1f379e into main May 8, 2024
6 checks passed
@mamullen13316 mamullen13316 deleted the services_update branch May 8, 2024 19:40
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.

None yet

1 participant