python-redsys 1.2.0__tar.gz → 1.2.1__tar.gz

{python_redsys-1.2.0 → python_redsys-1.2.1}/PKG-INFO

@@ -1,11 +1,11 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.4
 Name: python-redsys
-Version: 1.2.0
+Version: 1.2.1
 Summary: A simple, clean and less dependant client to handle payments through Redsys.
-Home-page: https://github.com/systemallica/python-redsys
 License: MIT
-Author: Andrés Reverón Molina
-Author-email: andres@reveronmolina.me
+License-File: LICENSE
+Author: David Díaz
+Author-email: d.diazp@gmail.com
 Requires-Python: >=3.8,<4.0
 Classifier: Intended Audience :: Developers
 Classifier: License :: OSI Approved :: MIT License
@@ -15,7 +15,12 @@ Classifier: Programming Language :: Python :: 3.8
 Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
+Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
+Classifier: Programming Language :: Python :: 3.15
 Requires-Dist: pycryptodome (>=3.19.0,<4.0.0)
+Project-URL: Homepage, https://github.com/ddiazpinto/python-redsys
 Description-Content-Type: text/markdown
 
 

{python_redsys-1.2.0 → python_redsys-1.2.1}/pyproject.toml

@@ -1,11 +1,11 @@
 [tool.poetry]
 name = "python-redsys"
-version = "1.2.0"
+version = "1.2.1"
 description = "A simple, clean and less dependant client to handle payments through Redsys."
-authors = ["Andrés Reverón Molina <andres@reveronmolina.me>", "David Díaz <d.diazp@gmail.com>"]
+authors = ["David Díaz <d.diazp@gmail.com>", "Andrés Reverón Molina <andres@reveronmolina.me>"]
 license = "MIT"
 readme = "README.md"
-homepage = "https://github.com/systemallica/python-redsys"
+homepage = "https://github.com/ddiazpinto/python-redsys"
 classifiers = [
         'Intended Audience :: Developers',
         'License :: OSI Approved :: MIT License',

{python_redsys-1.2.0 → python_redsys-1.2.1}/redsys/__init__.py

@@ -1,2 +1,2 @@
-__version__ = "1.2.0"
+__version__ = "1.2.1"
 __version_info__ = tuple(int(num) if num.isdigit() else num for num in __version__.replace("-", ".", 1).split("."))

{python_redsys-1.2.0 → python_redsys-1.2.1}/redsys/request.py

@@ -77,9 +77,11 @@ class Request:
     their values according to the platform specifications
     """
 
-    _parameters: Dict[str, Any] = {}
+    _parameters: Dict[str, Any]
 
     def __init__(self, parameters: Dict[str, Any]) -> None:
+        # __setattr__ only handles merchant parameters, so the per-instance store is set directly.
+        object.__setattr__(self, "_parameters", {})
         for key, value in parameters.items():
             if key in MERCHANT_PARAMETERS_MAP:
                 if check := getattr(self, f"check_{str(key)}", None):
@@ -90,7 +92,7 @@ class Request:
 
     def __getattr__(self, item: str) -> Any:
         if item in MERCHANT_PARAMETERS_MAP:
-            return self._parameters[item]
+            return self._parameters.get(item)
 
     def __setattr__(self, key, value):
         if key in MERCHANT_PARAMETERS_MAP:

{python_redsys-1.2.0 → python_redsys-1.2.1}/redsys/response.py

@@ -93,9 +93,11 @@ class Response:
     Defines a response
     """
 
-    _parameters: Dict[str, Any] = {}
+    _parameters: Dict[str, Any]
 
     def __init__(self, parameters: Dict[str, Any]):
+        # __setattr__ only handles merchant parameters, so the per-instance store is set directly.
+        object.__setattr__(self, "_parameters", {})
         MERCHANT_PARAMETERS_MAP_REVERSE = {value: key for key, value in MERCHANT_PARAMETERS_MAP.items()}
         for key, value in parameters.items():
             reversed_parameter = MERCHANT_PARAMETERS_MAP_REVERSE.get(key, key)
@@ -104,7 +106,7 @@ class Response:
 
     def __getattr__(self, item: str) -> Any:
         if item in MERCHANT_PARAMETERS_MAP:
-            return self._parameters[item]
+            return self._parameters.get(item)
 
     def __setattr__(self, key: str, value: Any):
         if key in MERCHANT_PARAMETERS_MAP:

{python_redsys-1.2.0 → python_redsys-1.2.1}/redsys/tests/test_request.py

@@ -63,3 +63,12 @@ class TestRequest:
         merchant_url = "".join(choice("abcdefghijklmnopqrtsuvwxyz-0123456789") for _ in range(251))
         with pytest.raises(ValueError):
             assert Request.check_merchant_url(merchant_url)
+
+    def test_requests_do_not_share_parameters(self):
+        first = Request(
+            {"order": "000000000001", "amount": D("1.00"), "currency": EUR, "mobile_number": "+34700000000"}
+        )
+        second = Request({"order": "000000000002", "amount": D("2.00"), "currency": EUR})
+        assert "Ds_Merchant_Bizum_MobileNumber" not in second.prepare_parameters()
+        assert second.mobile_number is None
+        assert first.prepare_parameters()["Ds_Merchant_Bizum_MobileNumber"] == "+34700000000"

{python_redsys-1.2.0 → python_redsys-1.2.1}/redsys/tests/test_response.py

@@ -1,3 +1,5 @@
+from decimal import Decimal
+
 from redsys.response import Response
 
 
@@ -21,3 +23,10 @@ class TestResponse:
         assert response.is_paid is True
         assert response.is_canceled is False
         assert response.is_refunded is False
+
+    def test_responses_do_not_share_parameters(self):
+        first = Response({"Ds_Response": "0000", "Ds_Order": "000000000001", "Ds_Amount": "100"})
+        second = Response({"Ds_Response": "0190", "Ds_Order": "000000000002"})
+        assert "amount" not in second.parameters
+        assert second.amount is None
+        assert first.amount == Decimal("1.00")