Projet

Général

Profil

Bug #8042

expose lasso_provider_verify_signature()

Ajouté par Florian Best il y a plus de 3 ans. Mis à jour il y a environ 3 ans.

Statut:
Nouveau
Priorité:
Normal
Assigné à:
-
Catégorie:
Core
Version cible:
Début:
12 août 2015
Echéance:
% réalisé:

0%

Patch proposed:
Oui

Description

The function expose lasso_provider_verify_signature() is currently not exposed in the symbol table.
As projects like crudesaml (http://ftp.espci.fr/pub/crudesaml/README) uses this function it would be nice to expose it.
The attached patch does this.

lasso.patch Voir (2,83 ko) Florian Best, 12 août 2015 15:47

Historique

#1 Mis à jour par Florian Best il y a plus de 3 ans

I just found a related problem: http://www.sogo.nu/bugs/view.php?id=2958

#2 Mis à jour par Benjamin Dauvergne il y a plus de 3 ans

  • Version cible mis à future

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

  • Version cible changé de future à 2.5.1

#4 Mis à jour par Benjamin Dauvergne il y a plus de 3 ans

Did you test your patch ? It seems to me that for supporting the xmlDoc type in the bindings (there are more than one, your patch lacks Perl, Java and PHP support) a lot more has to be done. There should be at least a conversion function from string to xmlDoc (as there is one from string to xmlNode).

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

To integrate this patch I'll need to know the author and he must state that the patch is contributed under the MIT license (it's our policy to keep our capacity to relicense Lasso).

#7 Mis à jour par Florian Best il y a plus de 3 ans

Yes, I can't speak french but from what https://wiki.auf.org/wikiteki/Projet/SOGo/TestsSAML?action=info says it seems the patch is created by https://wiki.auf.org/wikiteki/MoussaNombr%C3%A9?action=show&redirect=MoussaNombre
I send a message to this email address hoping that it exists and he/she will release the patch under MIT license.

#8 Mis à jour par Benjamin Dauvergne il y a environ 3 ans

  • Version cible changé de 2.5.1 à future

#9 Mis à jour par Florian Best il y a environ 3 ans

Nobody is answering, it seems not possible to contact the owner of the patch. Is it really a so hard issue to just "reimplement" these some lines?

#10 Mis à jour par Benjamin Dauvergne il y a environ 3 ans

Maybe not, try it :)

Formats disponibles : Atom PDF