Projet

Général

Profil

Bug #18771

xmlsec/soap.h replacement

Ajouté par Kranium Gikos Mendoza il y a plus de 6 ans. Mis à jour il y a presque 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
Début:
18 septembre 2017
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

xmlsec 1.2.25 has deprecated xmlsec/soap.h and plans to completely remove it in +2 releases.

https://github.com/lsh123/xmlsec/issues/86
https://github.com/lsh123/xmlsec/pull/105

The soap headers are now disabled in xmlsec by default and distros will have to compile with --enable-soap to enable it back.

Raising this ticket to request finding a replacement for xmlsec/soap.h


Fichiers


Demandes liées

Dupliqué par Lasso - Bug #21699: Lasso no longer compiles with xmlsec1 built with default flags.Rejeté08 février 2018

Actions

Révisions associées

Révision 1d56cd1e (diff)
Ajouté par Benjamin Dauvergne il y a environ 6 ans

replace use of <xmlsec/soap.h> which is deprecated (fixes #18771)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

We really don't use it at all:

lasso/id-wsf/wsf_profile.c:     header = xmlSecSoap11GetHeader(envelope);
lasso/id-wsf/wsf_profile.c:     body = xmlSecSoap11GetBody(envelope);
lasso/xml/tools.c:      return xmlSecCheckNodeName(root, xmlSecNodeEnvelope, xmlSecSoap11Ns) ||
lasso/xml/tools.c:              xmlSecCheckNodeName(root, xmlSecNodeEnvelope, xmlSecSoap12Ns);
lasso/xml/tools.c:      is_soap11 = xmlSecCheckNodeName(root, xmlSecNodeEnvelope, xmlSecSoap11Ns);
lasso/xml/tools.c:      is_soap12 = xmlSecCheckNodeName(root, xmlSecNodeEnvelope, xmlSecSoap12Ns);
lasso/xml/tools.c:                      body = xmlSecSoap11GetBody(root);
lasso/xml/tools.c:                      body = xmlSecSoap12GetBody(root);

Can you contribute replacement for these line of code ?

I think it would also be the good time to completely remove the incomplete and useless ID-WSF support.

#2

Mis à jour par Kranium Gikos Mendoza il y a plus de 6 ans

Apologies for the latency @Benjamin

I am not sure if I have time and expertise in contributing into the c++. I only came across this issue since I package the lasso and mod_auth_mellon in NixOS.

#3

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

Hi, no problem I did it, could you test the following patch and confirm it fixes this problem ?

#4

Mis à jour par Kranium Gikos Mendoza il y a plus de 6 ans

I'm afraid the patch appears didn't pick up lasso/xml/xmlsec_soap.h when it was prepared

make[5]: Leaving directory '/tmp/nix-build-lasso-2.5.1.drv-0/lasso-2.5.1/lasso/xml/ecp'
make[5]: Entering directory '/tmp/nix-build-lasso-2.5.1.drv-0/lasso-2.5.1/lasso/xml'
CC tools.lo
tools.c:73:35: fatal error: lasso/xml/xmlsec_soap.h: No such file or directory
#include <lasso/xml/xmlsec_soap.h>
^
compilation terminated.
make[5]: *** [Makefile:1054: tools.lo] Error 1

#6

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

I'll add copyright mentions about xmlsec later (but code is so succint and so evident I don't know if it's mandatory, will do it only because xmlsec is a fine project).

#7

Mis à jour par Thomas Noël il y a environ 6 ans

  • Dupliqué par Bug #21699: Lasso no longer compiles with xmlsec1 built with default flags. ajouté
#8

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

  • Statut changé de Nouveau à Résolu (à déployer)
  • % réalisé changé de 0 à 100
#9

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Assigné à mis à John Dennis
  • Version cible mis à 2.6.0
#10

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF