Make params in HSETEX mutual exclusive

Signed-off-by: Joshua <joshua.gehlen@fkie.fraunhofer.de>

Author: Joshua

tests/test_commands.py

@@ -3243,6 +3243,11 @@ def test_hsetex(self, r):
         assert r.hsetex("a", "field1", "value2", ex=5) == 1
         assert r.hget("a", "field1") == b"value2"
 
+    @skip_if_server_version_lt("9.0.0")
+    def test_hset_ex_invalid_params(self, r):
+        with pytest.raises(exceptions.DataError):
+            r.hsetex("a", "field1", "value1", ex=5, px=5000)  # Both ex and px provided
+
     @skip_if_server_version_lt("9.0.0")
     def test_hsetex_px(self, r):
         assert r.hsetex("a", "field1", "value1", px=5000) == 1

valkey/commands/core.py

@@ -5066,6 +5066,7 @@ def hsetex(
         px: Union[ExpiryT, None] = None,
         exat: Union[AbsExpiryT, None] = None,
         pxat: Union[AbsExpiryT, None] = None,
+        keepttl: bool = False,
         nx: bool = False,
         xx: bool = False,
         fnx: bool = False,
@@ -5082,6 +5083,15 @@ def hsetex(
 
         if key is None and not mapping and not items:
             raise DataError("'hsetex' with no key value pairs")
+
+        if int(keepttl) + sum(arg is not None for arg in [ex, px, exat, pxat]) > 1:
+            raise DataError(
+                "Only one of 'ex', 'px', 'exat', 'pxat', or 'keepttl' can be specified."
+            )
+        if nx and xx:
+            raise DataError("Only one of 'nx' or 'xx' can be specified.")
+        if fnx and fxx:
+            raise DataError("Only one of 'fnx' or 'fxx' can be specified.")
         pieces = []
         if ex is not None:
             pieces.extend(["EX", ex])
@@ -5107,6 +5117,8 @@ def hsetex(
         if mapping:
             pieces.append(len(mapping))
             for key, value in mapping.items():
+                if key is None or value is None:
+                    raise DataError("'hsetex' mapping contains None key or value")
                 pieces.append(key)
                 pieces.append(value)
         if items: