From e0e9a6e5b2ee6dd6254c9b37e06a3102034d0c54 Mon Sep 17 00:00:00 2001
From: chkp-edenbr <94056191+chkp-edenbr@users.noreply.github.com>
Date: Sun, 10 Dec 2023 13:21:49 +0200
Subject: [PATCH] v5.1.3 (#116)

---
 CHANGELOG.rst                                | 25 ++++++++++++++++++++
 changelogs/changelog.yaml                    | 18 ++++++++++++++
 galaxy.yml                                   |  2 +-
 plugins/httpapi/checkpoint.py                |  3 ++-
 plugins/modules/cp_mgmt_get_interfaces.py    |  7 +++---
 plugins/modules/cp_mgmt_vsx_run_operation.py |  3 ++-
 6 files changed, 51 insertions(+), 7 deletions(-)

diff --git a/CHANGELOG.rst b/CHANGELOG.rst
index 2d86a48..b8dd642 100644
--- a/CHANGELOG.rst
+++ b/CHANGELOG.rst
@@ -4,6 +4,31 @@ Check_Point.Mgmt Release Notes
 
 .. contents:: Topics
 
+v5.1.3
+======
+
+Release Summary
+---------------
+
+This is release 5.1.3 of ``check_point.mgmt``, released on 2023-12-13.
+
+Bugfixes
+--------
+
+- httpapi/checkpoint.py - Raise a fatal error if login wasn't successful.
+
+v5.1.2
+======
+
+Release Summary
+---------------
+
+This is release 5.1.2 of ``check_point.mgmt``, released on 2023-12-12.
+
+Minor Changes
+--------
+
+- meta/runtime.yml - update minimum Ansible version required to 2.14.0.
 
 v5.1.1
 ======
diff --git a/changelogs/changelog.yaml b/changelogs/changelog.yaml
index 047a4af..3a87b8d 100644
--- a/changelogs/changelog.yaml
+++ b/changelogs/changelog.yaml
@@ -1035,3 +1035,21 @@ releases:
     fragments:
     - 5.1.1.yml
     release_date: '2022-07-17'
+  5.1.2:
+    changes:
+      minor_changes:
+      - meta/runtime.yml - update minimum Ansible version required to 2.14.0.
+      release_summary: This is release 5.1.2 of ``check_point.mgmt``, released on
+        2023-12-12.
+    fragments:
+    - 5.1.2.yml
+    release_date: '2022-12-12'
+  5.1.3:
+    changes:
+      bugfixes:
+      - httpapi/checkpoint.py - Raise a fatal error if login wasn't successful.
+      release_summary: This is release 5.1.3 of ``check_point.mgmt``, released on
+        2023-12-13.
+    fragments:
+    - 5.1.2.yml
+    release_date: '2022-12-13'
\ No newline at end of file
diff --git a/galaxy.yml b/galaxy.yml
index 3811b28..d48de32 100644
--- a/galaxy.yml
+++ b/galaxy.yml
@@ -9,7 +9,7 @@ namespace: check_point
 name: mgmt
 
 # The version of the collection. Must be compatible with semantic versioning
-version: 5.1.1
+version: 5.1.3
 
 # The path to the Markdown (.md) readme file. This path is relative to the root of the collection
 readme: README.md
diff --git a/plugins/httpapi/checkpoint.py b/plugins/httpapi/checkpoint.py
index 7c51e4a..606af77 100644
--- a/plugins/httpapi/checkpoint.py
+++ b/plugins/httpapi/checkpoint.py
@@ -67,7 +67,8 @@ def login(self, username, password):
             )
         url = "/web_api/login"
         response, response_data = self.send_request(url, payload)
-
+        if response != 200:
+            raise ConnectionError("Login to server failed: %s" % response_data)
         try:
             self.connection._auth = {"X-chkp-sid": response_data["sid"]}
         except KeyError:
diff --git a/plugins/modules/cp_mgmt_get_interfaces.py b/plugins/modules/cp_mgmt_get_interfaces.py
index a93d986..ad40a78 100644
--- a/plugins/modules/cp_mgmt_get_interfaces.py
+++ b/plugins/modules/cp_mgmt_get_interfaces.py
@@ -31,10 +31,9 @@
 short_description: Get physical interfaces with or without their topology from a Gaia Security Gateway or Cluster.
 description:
   - Get physical interfaces with or without their topology from a Gaia Security Gateway or Cluster.
-Note: The fetched topology is based on static routes.
-Prerequisites:
-- SIC must be established in the Security Gateway or Cluster Member object.
-- Security Gateway or Cluster Members must be up and running.
+  - The fetched topology is based on static routes.
+  - SIC must be established in the Security Gateway or Cluster Member object.
+  - Security Gateway or Cluster Members must be up and running.
   - All operations are performed over Web Services API.
 version_added: "5.0.0"
 author: "Eden Brillant (@chkp-edenbr)"
diff --git a/plugins/modules/cp_mgmt_vsx_run_operation.py b/plugins/modules/cp_mgmt_vsx_run_operation.py
index b22f342..15db77e 100644
--- a/plugins/modules/cp_mgmt_vsx_run_operation.py
+++ b/plugins/modules/cp_mgmt_vsx_run_operation.py
@@ -30,7 +30,8 @@
 module: cp_mgmt_vsx_run_operation
 short_description: Run the VSX operation by its name and parameters.
 description:
-  - Run the VSX operation by its name and parameters. <br><b>Important note:</b> An automatic session publish is part of all the operations in this API.
+  - Run the VSX operation by its name and parameters.
+  - An automatic session publish is part of all the operations in this API.
   - All operations are performed over Web Services API.
 version_added: "5.0.0"
 author: "Eden Brillant (@chkp-edenbr)"