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

Add support for PanOS XML configuration backup #2360

Closed
wants to merge 2 commits into from

Conversation

pv2b
Copy link
Contributor

@pv2b pv2b commented Sep 3, 2021

Pre-Request Checklist

  • Passes rubocop code analysis (try rubocop --auto-correct)
  • Tests added or adapted (try rake test)
  • Changes are reflected in the documentation
  • User-visible changes appended to CHANGELOG.md

Description

This solves #440 by adding functionality to back up Palo Alto firewall XML configuration files using the HTTPS API.

@codecov-commenter
Copy link

codecov-commenter commented Sep 3, 2021

Codecov Report

Merging #2360 (cd6031a) into master (8179f48) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2360   +/-   ##
=======================================
  Coverage   62.59%   62.59%           
=======================================
  Files          30       30           
  Lines        1497     1497           
=======================================
  Hits          937      937           
  Misses        560      560           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8179f48...cd6031a. Read the comment docs.

@mortzu
Copy link
Collaborator

mortzu commented Jan 26, 2022

Please process the checklist

@mortzu mortzu closed this May 13, 2022
@sts sts mentioned this pull request Apr 5, 2023
4 tasks
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

3 participants