Project

General

Profile

Development #71399

Dépréciation de xmlSecBase64Decode

Added by Benjamin Dauvergne 2 months ago. Updated 2 months ago.

Status:
Résolu (à déployer)
Priority:
Normal
Category:
-
Target version:
-
Start date:
17 November 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

La fonction xmlSecBase64Decode a été jugée mal conçu par le développeur d'xmlsec, elle est donc dépréciée, on remplacera tous ses usages par une API interne lasso_base64_decode elle même basée sur la nouvelle fonction xmlSecBase64Decode_ex introduite à partir de xmlsec 1.2.35. Un nouveau #define permettra de conserver une compatibilité avec xmlsec < 1.2.35.


Files

Associated revisions

Revision 228ac947 (diff)
Added by Benjamin Dauvergne 2 months ago

Make lasso_inflate output the inflated buffer size (#71399)

Revision b263815f (diff)
Added by Benjamin Dauvergne 2 months ago

Add new define LASSO_XMLSEC_VERSION_NUMBER allow version check on libxmlsec (#71399)

Revision 1aa6271f (diff)
Added by Benjamin Dauvergne 2 months ago

Adapt lasso_base64_decode to the deprecation of xmlSecBase64Decode (#71399)

We now use the non-deprecated new API (since xmlsec 1.2.35) xmlSecBase64Decode_ex.

Revision 8a588a8a (diff)
Added by Benjamin Dauvergne 2 months ago

Replace all use of xmlSecBase64Decode by lasso_base64_decode (#71399)

History

#2

Updated by Pierre Ducroquet 2 months ago

Je ne comprends pas comment le patch 0001 passe.
On rajoute un paramètre à la fonction lasso_inflate, mais je ne vois pas de patch des appels à cette fonction pour ajouter le paramètre. Du coup je pense qu'il y a un truc invalide.

#3

Updated by Benjamin Dauvergne 2 months ago

Pierre Ducroquet a écrit :

Je ne comprends pas comment le patch 0001 passe.
On rajoute un paramètre à la fonction lasso_inflate, mais je ne vois pas de patch des appels à cette fonction pour ajouter le paramètre. Du coup je pense qu'il y a un truc invalide.

Ces modifications se sont glissées dans le patch 0004, je vais voir pour rapatrier ça, mais c'est pas sûr.

#4

Updated by Pierre Ducroquet 2 months ago

  • Status changed from Solution proposée to Solution validée

Si c'est dans les habitudes de développement je valide.

#5

Updated by Benjamin Dauvergne 2 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 8a588a8acb4a9cb7c7cb4dfd91a8278264a6d15a
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Wed Nov 16 15:40:19 2022 +0100

    Replace all use of xmlSecBase64Decode by lasso_base64_decode (#71399)

commit 1aa6271f93e48b24f42991aba8906dfd073a1fe3
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Wed Nov 16 15:38:35 2022 +0100

    Adapt lasso_base64_decode to the deprecation of xmlSecBase64Decode (#71399)

    We now use the non-deprecated new API (since xmlsec 1.2.35) xmlSecBase64Decode_ex.

commit b263815fa495371adc1bb59045cf0c33fa3fb71a
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Wed Nov 16 13:08:04 2022 +0100

    Add new define LASSO_XMLSEC_VERSION_NUMBER allow version check on libxmlsec (#71399)

commit 228ac9470f383c5c9dad9f4d314b708c9fd45f16
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Wed Nov 16 15:36:53 2022 +0100

    Make lasso_inflate output the inflated buffer size (#71399)

Also available in: Atom PDF