From 595aeaa2651ba0a1f80565ceac7550fd1abd38fe Mon Sep 17 00:00:00 2001
From: Amin Solhizadeh <amin.solhizadeh@gmail.com>
Date: Wed, 13 Mar 2024 16:46:55 +0100
Subject: [PATCH] Make the library backward compatible

Starting from version xmlsec1-1.3.3, some of the constant and keys that were
deprecated, removed from the library entirely. This change would add
version awareness and backward compatible.
---
 doc/source/modules/constants.rst |  6 +++++-
 src/constants.c                  |  7 +++++--
 src/keys.c                       | 15 ++++++++++++---
 src/xmlsec/constants.pyi         |  1 +
 4 files changed, 23 insertions(+), 6 deletions(-)

diff --git a/doc/source/modules/constants.rst b/doc/source/modules/constants.rst
index 8fdac11..3df6b50 100644
--- a/doc/source/modules/constants.rst
+++ b/doc/source/modules/constants.rst
@@ -47,9 +47,13 @@ KeyData
 
    The DSA key klass.
 
+.. data:: xmlsec.constants.KeyDataEcdsa
+
+   (Deprecated. The EC key klass) The ECDSA key klass.
+
 .. data:: xmlsec.constants.KeyDataEc
 
-   The ECDSA key klass.
+   The EC key klass.
 
 .. data:: xmlsec.constants.KeyDataHmac
 
diff --git a/src/constants.c b/src/constants.c
index f2c5a63..bd1fa5e 100644
--- a/src/constants.c
+++ b/src/constants.c
@@ -445,8 +445,11 @@ int PyXmlSec_ConstantsModule_Init(PyObject* package) {
 #ifndef XMLSEC_NO_DSA
     PYXMLSEC_ADD_KEYDATA_CONSTANT(KeyDataDsa, "DSA")
 #endif
-#if XMLSEC_VERSION_HEX > 0x10212
-    // from version 1.2.19
+#if XMLSEC_VERSION_HEX > 0x10212 && XMLSEC_VERSION_HEX < 0x10303
+    // from version 1.2.19 to version 1.3.2 (inclusive)
+    PYXMLSEC_ADD_KEYDATA_CONSTANT(KeyDataEcdsa, "ECDSA")
+#elif XMLSEC_VERSION_HEX >= 0x10303
+    // from version 1.3.3 (inclusive)
     PYXMLSEC_ADD_KEYDATA_CONSTANT(KeyDataEc, "ECDSA")
 #endif
     PYXMLSEC_ADD_KEYDATA_CONSTANT(KeyDataHmac, "HMAC")
diff --git a/src/keys.c b/src/keys.c
index 8b84c34..5ff04aa 100644
--- a/src/keys.c
+++ b/src/keys.c
@@ -163,7 +163,12 @@ static PyObject* PyXmlSec_KeyFromFile(PyObject* self, PyObject* args, PyObject*
     if (is_content) {
         key->handle = xmlSecCryptoAppKeyLoadMemory((const xmlSecByte*)data, (xmlSecSize)data_size, format, password, NULL, NULL);
     } else {
-        key->handle = xmlSecCryptoAppKeyLoadEx(data, xmlSecKeyDataTypePrivate, format, password, NULL, NULL);
+        #if XMLSEC_VERSION_HEX >= 0x10303
+            // from version 1.3.3 (inclusive)
+            key->handle = xmlSecCryptoAppKeyLoadEx(data, xmlSecKeyDataTypePrivate, format, password, NULL, NULL);
+        #else
+            key->handle = xmlSecCryptoAppKeyLoad(data, format, password, NULL, NULL);
+        #endif
     }
     Py_END_ALLOW_THREADS;
 
@@ -206,8 +211,12 @@ static PyObject* PyXmlSec_KeyFromEngine(PyObject* self, PyObject* args, PyObject
     if ((key = PyXmlSec_NewKey1((PyTypeObject*)self)) == NULL) goto ON_FAIL;
 
     Py_BEGIN_ALLOW_THREADS;
-    key->handle = xmlSecCryptoAppKeyLoadEx(engine_and_key_id, xmlSecKeyDataTypePrivate, xmlSecKeyDataFormatEngine, NULL, xmlSecCryptoAppGetDefaultPwdCallback(),
-                                         (void*)engine_and_key_id);
+    #if XMLSEC_VERSION_HEX >= 0x10303
+        // from version 1.3.3 (inclusive)
+        key->handle = xmlSecCryptoAppKeyLoadEx(engine_and_key_id, xmlSecKeyDataTypePrivate, xmlSecKeyDataFormatEngine, NULL, xmlSecCryptoAppGetDefaultPwdCallback(), (void*)engine_and_key_id);
+    #else
+        key->handle = xmlSecCryptoAppKeyLoad(engine_and_key_id, xmlSecKeyDataFormatEngine, NULL, xmlSecCryptoAppGetDefaultPwdCallback(), (void*)engine_and_key_id);
+    #endif
     Py_END_ALLOW_THREADS;
 
     if (key->handle == NULL) {
diff --git a/src/xmlsec/constants.pyi b/src/xmlsec/constants.pyi
index 80afdd2..3c3ea94 100644
--- a/src/xmlsec/constants.pyi
+++ b/src/xmlsec/constants.pyi
@@ -30,6 +30,7 @@ KeyDataAes: Final[__KeyData]
 KeyDataDes: Final[__KeyData]
 KeyDataDsa: Final[__KeyData]
 KeyDataEc: Final[__KeyData]
+KeyDataEcdsa: Final[__KeyData]
 KeyDataEncryptedKey: Final[__KeyData]
 KeyDataFormatBinary: Final[int]
 KeyDataFormatCertDer: Final[int]