@devlas/dte-sii 2.12.19 → 2.12.21

package/EnviadorSII.js

@@ -1174,7 +1174,11 @@ class EnviadorSII {
     log.log('XML Length:', xml.length, 'bytes');
 
     const [rutNum, dv] = rutEmisor.split('-');
-    const [rutEnviaNum, dvEnvia] = this.certificado.rut.split('-');
+    // Fallback a rutEmisor si el certificado no expone RUT (ver
+    // docs/EXTRACCION_RUT_CERTIFICADO.md) — evita un TypeError por .split
+    // sobre null cuando el PFX no tiene el RUT en ningún campo conocido.
+    const rutEnvia = this.certificado.rut || rutEmisor;
+    const [rutEnviaNum, dvEnvia] = rutEnvia.split('-');
     
     const boundary = '----WebKitFormBoundary' + Math.random().toString(36).substring(2);
     

package/SiiCertificacion.js

@@ -1281,6 +1281,13 @@ class SiiCertificacion {
       const formAction = SiiSession.extractFormAction(body);
       if (formAction && formAction.includes('pe_avance9')) {
         const hiddenPaso3 = SiiSession.extractInputValues(body);
+        // El HTML tiene DOS botones (CONFIRMAR="Confirmar Declaración" y Salir="No Confirmar",
+        // este último con TYPE=button). extractInputValues lee <input> sin distinguir el type,
+        // así que Salir queda en hiddenPaso3 con valor "No Confirmar". Un browser real solo
+        // envía el par name/value del botón que el usuario clickeó — nunca ambos. Si se manda
+        // Salir=No Confirmar junto con CONFIRMAR, el SII podría interpretar la presencia de ese
+        // campo como "el usuario canceló", pisando la confirmación real.
+        delete hiddenPaso3.Salir;
         const opcFields = {};
         for (const key of Object.keys(hiddenPaso3)) {
           if (/^OPC\d+$/.test(key)) opcFields[key] = 'S';

package/cert/BoletaCert.js

@@ -214,7 +214,7 @@ class BoletaCert {
     
     const envioBoleta = new EnvioBOLETA({
       rutEmisor: this.emisor.rut,
-      rutEnvia: this.certificado.rut,
+      rutEnvia: this.certificado.rut || this.emisor.rut,
       fchResol: this.resolucion.fecha,
       nroResol: this.resolucion.numero,
       certificado: this.certificado,
@@ -228,7 +228,7 @@ class BoletaCert {
     // Carátula para certificación
     envioBoleta.setCaratula({
       RutEmisor: this.emisor.rut,
-      RutEnvia: this.certificado.rut,
+      RutEnvia: this.certificado.rut || this.emisor.rut,
       RutReceptor: '60803000-K', // SII en certificación
       FchResol: this.resolucion.fecha,
       NroResol: this.resolucion.numero,
@@ -291,7 +291,7 @@ class BoletaCert {
     // Establecer carátula
     consumoFolio.setCaratula({
       RutEmisor: this.emisor.rut,
-      RutEnvia: this.certificado.rut,
+      RutEnvia: this.certificado.rut || this.emisor.rut,
       FchResol: this.resolucion.fecha,
       NroResol: this.resolucion.numero,
       SecEnvio: options?.secEnvio || 1,
@@ -358,7 +358,7 @@ class BoletaCert {
     
     consumoFolio.setCaratula({
       RutEmisor: this.emisor.rut,
-      RutEnvia: this.certificado.rut,
+      RutEnvia: this.certificado.rut || this.emisor.rut,
       FchResol: this.resolucion.fecha,
       NroResol: this.resolucion.numero,
       SecEnvio: options.secEnvio,

package/package.json

@@ -1,12 +1,15 @@
 {
   "name": "@devlas/dte-sii",
-  "version": "2.12.19",
+  "version": "2.12.21",
   "description": "Facturación y boletas electrónicas para el SII de Chile. Genera, timbra, firma y envía DTEs, libros electrónicos y automatiza la certificación.",
   "main": "index.js",
   "types": "dte-sii.d.ts",
   "license": "MIT",
   "author": "Devlas SpA <ti@devlas.cl> (https://devlas.cl)",
   "homepage": "https://github.com/devlas-cl/dte-sii",
+  "scripts": {
+    "test": "node test/pfx.test.js"
+  },
   "repository": {
     "type": "git",
     "url": "https://github.com/devlas-cl/dte-sii.git"

package/utils/pfx.js

@@ -179,11 +179,48 @@ function extractSubjectFields(cert) {
 }
 
 /**
- * Extraer RUT del certificado (desde serialNumber, OU o CN)
+ * Extraer RUT desde el Subject Alternative Name (SAN) con OID
+ * 1.3.6.1.4.1.8321.1 — reservado bajo la normativa chilena de firma
+ * electrónica (Ley 19.799). A diferencia de serialNumber/OU/CN, que son
+ * convención de cada CA, este OID es consistente entre CAs (verificado
+ * contra Acepta, IDOK y Signapis — ver docs/EXTRACCION_RUT_CERTIFICADO.md).
+ * @param {forge.pki.Certificate} cert - Certificado
+ * @returns {string|null} RUT o null
+ */
+function extractRutFromSan(cert) {
+  const san = cert.getExtension('subjectAltName');
+  if (!san || !san.altNames) return null;
+
+  for (const alt of san.altNames) {
+    if (alt.type !== 0) continue; // 0 = otherName
+    try {
+      const oid = forge.asn1.derToOid(alt.value[0].value);
+      if (oid !== '1.3.6.1.4.1.8321.1') continue;
+
+      const inner = alt.value[1] && alt.value[1].value && alt.value[1].value[0];
+      if (!inner || !inner.value) continue;
+
+      const clean = inner.value.replace(/\./g, '').toUpperCase();
+      if (/^\d{7,8}-[\dK]$/.test(clean)) {
+        return clean;
+      }
+    } catch {
+      // OID no parseable en este altName — seguir con el resto y caer al fallback
+    }
+  }
+
+  return null;
+}
+
+/**
+ * Extraer RUT del certificado (desde SAN OID, serialNumber, OU o CN)
  * @param {forge.pki.Certificate} cert - Certificado
  * @returns {string|null} RUT o null
  */
 function extractRutFromCertificate(cert) {
+  const sanRut = extractRutFromSan(cert);
+  if (sanRut) return sanRut;
+
   const subject = extractSubjectFields(cert);
 
   // Intentar desde serialNumber (más confiable)