Merge pull request #74 from aruba/v2.0a14-2

v2.0a14-2

Author: KarthikSKumar98

docs/index.md

@@ -26,7 +26,7 @@ Today, there are two versions of PyCentral, each designed for different versions
 | Version                                                        | Supports                                                                                       | Notes                        |
 | :------------------------------------------------------------- | :--------------------------------------------------------------------------------------------- | :--------------------------- |
 | [v1](https://pypi.org/project/pycentral/)                      | HPE Aruba Networking Central (Classic Central)                                                 | Legacy Version               |
-| [v2(pre-release)](https://pypi.org/project/pycentral/2.0a13/) | HPE Aruba Networking Central(new Central), GLP, HPE Aruba Networking Central (Classic Central) | Backwards compatible with v1 |
+| [v2(pre-release)](https://pypi.org/project/pycentral/2.0a14/) | HPE Aruba Networking Central(new Central), GLP, HPE Aruba Networking Central (Classic Central) | Backwards compatible with v1 |
 
 ## Quick Example
 

pycentral/new_monitoring/clients.py

@@ -1,6 +1,7 @@
 from ..utils.monitoring_utils import (
     execute_get,
     build_timestamp_filter,
+    _validate_mac_address,
 )
 from ..exceptions import ParameterError
 
@@ -382,6 +383,27 @@ def get_top_n_site_clients(
 
         return execute_get(central_conn, endpoint=path, params=params)
 
+    @staticmethod
+    def get_client_details(
+        central_conn,
+        client_mac,
+    ):
+        """
+        Fetch details for a specific client by MAC address.
+        This method makes an API call to the following endpoint - `GET network-monitoring/v1alpha1/clients/{client_mac}`
+
+        Args:
+            central_conn (NewCentralBase): Central connection object.
+            client_mac (str): MAC address of the client to query.
+
+        Returns:
+            (dict): Client details as returned by the API.
+        """
+        _validate_mac_address(client_mac)
+
+        path = f"clients/{client_mac}"
+        return execute_get(central_conn, endpoint=path)
+
     def _time_filter(params, start_time, end_time, duration):
         """
         Apply a time filter to params using unix timestamps.

pycentral/utils/monitoring_utils.py

@@ -3,6 +3,7 @@
 from pycentral.utils.url_utils import generate_url
 
 from ..exceptions import ParameterError
+import re
 
 
 def build_timestamp_filter(
@@ -37,17 +38,11 @@ def build_timestamp_filter(
 
     # --- Validation ---
     if (start_time or end_time) and duration:
-        raise ValueError(
-            "Cannot specify start/end timestamps together with duration."
-        )
+        raise ValueError("Cannot specify start/end timestamps together with duration.")
     if (start_time and not end_time) or (end_time and not start_time):
-        raise ValueError(
-            "Both start_time and end_time must be provided together."
-        )
+        raise ValueError("Both start_time and end_time must be provided together.")
     if not duration and not (start_time and end_time):
-        raise ValueError(
-            "Provide either both start_time and end_time or a duration."
-        )
+        raise ValueError("Provide either both start_time and end_time or a duration.")
 
     # --- Case 1: Start + End (pass-through) ---
     if start_time and end_time:
@@ -161,9 +156,7 @@ def simplified_site_resp(site):
         ),
     }
     site["alerts"] = {
-        "critical": site.get("alerts", {})
-        .get("groups", [{}])[0]
-        .get("count", 0)
+        "critical": site.get("alerts", {}).get("groups", [{}])[0].get("count", 0)
         if site.get("alerts", {}).get("groups")
         else 0,
         "total": site.get("alerts", {}).get("totalCount", 0),
@@ -185,11 +178,7 @@ def _groups_to_dict(groups_list):
     result = {"Poor": 0, "Fair": 0, "Good": 0}
     if isinstance(groups_list, list):
         for group in groups_list:
-            if (
-                isinstance(group, dict)
-                and "name" in group
-                and "value" in group
-            ):
+            if isinstance(group, dict) and "name" in group and "value" in group:
                 result[group["name"]] = group["value"]
     return result
 
@@ -246,3 +235,30 @@ def merged_dict_to_sorted_list(merged):
     except Exception:
         keys = sorted(merged.keys())
     return [{"timestamp": ts, **merged[ts]} for ts in keys]
+
+
+def _validate_mac_address(mac):
+    """
+    Validate a MAC address string and return True if valid.
+    Accepts the format AA:BB:CC:DD:EE:FF
+
+    Args:
+        mac (str): MAC address string to validate.
+
+    Returns:
+        (bool): True if the MAC address is valid.
+
+    Raises:
+        ParameterError: If mac is missing or does not match the expected format.
+
+    Note:
+        Internal SDK function
+    """
+    _MAC_PATTERN = re.compile(r"^([0-9A-Fa-f]{2}:){5}[0-9A-Fa-f]{2}$")
+    if not mac:
+        raise ParameterError("MAC address is required")
+    if not isinstance(mac, str) or not _MAC_PATTERN.match(mac):
+        raise ParameterError(
+            f"Invalid MAC address format: '{mac}'. Expected format: AA:BB:CC:DD:EE:FF"
+        )
+    return True

setup.py

@@ -5,7 +5,7 @@
 with open(path.join(this_directory, "README.md"), encoding="utf-8") as fh:
     long_description = fh.read()
 
-VERSION = "2.0a13"
+VERSION = "2.0a14"
 
 setuptools.setup(
     name="pycentral",
@@ -31,12 +31,12 @@
     ],
     python_requires=">=3.8",
     install_requires=[
-        "requests==2.32.4",
-        "PyYAML==6.0.2",
+        "requests==2.32.5",
+        "PyYAML==6.0.3",
         "oauthlib==3.2.2",
         "requests_oauthlib==2.0.0",
         "pytz==2025.2",
-        "protobuf==6.33.2",
+        "protobuf==6.33.5",
         "websocket-client==1.9.0",
     ],
     extras_require={"colorLog": ["colorlog"]},