Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[11.0][l10n_es_facturae] Validación certificado ssl www.facturae.es falla desde hace unos días #1069

Closed
adrian-pesol opened this issue Apr 22, 2019 · 6 comments
Milestone

Comments

@adrian-pesol
Copy link
Contributor

Hola @etobella , desde hace unos días "remote = urllib2.urlopen(sig_policy_identifier)" falla al validar el certificado de [http://www.facturae.es/politica_de_firma_formato_facturae/politica_de_firma_formato_facturae_v3_1.pdf] (Quizá tenga algo que ver con la renovación el 8 de abril de ese certificado) Sólo es necesario para calcular el hash y éste es conocido e invariable (Ohixl6upD6av8N7pEvDABhEL6hM=) por lo que para corregirlo en local hemos evitado obtener el pdf y calcular el hash cada vez y le pasamos el string del hash directamente:

Código anterior:
remote = urllib2.urlopen(sig_policy_identifier) etree.SubElement( sig_policy_hash, etree.QName(xmlsig.constants.DSigNs, 'DigestValue') ).text = base64.b64encode(hashlib.sha1(remote.read()).digest())

Corrección:
etree.SubElement( sig_policy_hash, etree.QName(xmlsig.constants.DSigNs, 'DigestValue') ).text = 'Ohixl6upD6av8N7pEvDABhEL6hM='

Es una corrección sencilla que hace que funcione por ahora, aunque no sé si pasar directamente el hash es lo más adecuado. ¿Qué opinas?

@pedrobaeza
Copy link
Member

Se ha estado comentando en #1052 (comment).

Si con esa corrección es suficiente, genial. Que confirme @etobella

@pedrobaeza pedrobaeza added this to the 11.0 milestone Apr 22, 2019
@etobella
Copy link
Member

Deberia valer

@rbetancor
Copy link

Es una corrección sencilla que hace que funcione por ahora, aunque no sé si pasar directamente el hash es lo más adecuado. ¿Qué opinas?

Que valer, vale y si tenemos en cuenta como funcionan muchas cosas de la administración electrónica en este país ... es probable que valga por mucho tiempo.

@pedrobaeza
Copy link
Member

Apuntadlo eso sí en known issues

@pedrobaeza
Copy link
Member

Hacéis el PR para arreglar la rama?

@pedrobaeza
Copy link
Member

Arreglado en #1071

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants