q-koa 13.8.37 → 13.8.39

package/core/file/plugins/weixin/controller.js

@@ -91,7 +91,13 @@ const sendWxFail = (ctx, msg = 'FAIL') => {
  * @param {string} options.md5Key     MD5 签名密钥（signType=MD5 时使用）
  * @returns {boolean}
  */
-const validateUnionNotifySign = ({ notifyData, signType, publicKey, md5Key }) => {
+const validateUnionNotifySign = ({
+  notifyData,
+  signType,
+  publicKey,
+  md5Key,
+  fallbackPlatformPublicKey,
+}) => {
   if (!notifyData || !notifyData.sign) return false
   const params = { ...notifyData }
   const sign = String(params.sign)
@@ -118,21 +124,50 @@ const validateUnionNotifySign = ({ notifyData, signType, publicKey, md5Key }) =>
     return expected === sign.toUpperCase()
   }
 
-  // 默认 RSA-SHA1（与 allinpay.js verifySign 保持一致）
-  if (!publicKey) return false
-  try {
-    // 兼容两种配置格式：已经带 PEM 头尾的直接用；纯 base64 body 再包裹头尾
-    const pem = String(publicKey).includes('-----BEGIN')
-      ? String(publicKey)
-      : `-----BEGIN PUBLIC KEY-----\n${publicKey}\n-----END PUBLIC KEY-----`
-    const verify = crypto.createVerify('RSA-SHA1')
-    verify.update(signStr, 'utf8')
-    return verify.verify(pem, sign, 'base64')
-  } catch (e) {
-    return false
+  // RSA 双算法兜底：先试 RSA-SHA256（通联 2048-bit 新平台证书），失败再退 RSA-SHA1（1024-bit 老平台证书）
+  // 两份证书的签名/密钥长度天然不同，1024-bit 公钥永远验不出 256 字节签名，2048-bit 公钥也验不出 128 字节签名，
+  // 所以顺序上即使「错算法试一次」也不会误判，只是多一次 CPU crypto 调用（微秒级，可忽略）。
+  // 尝试用户配置的 union_publicKey（兼容无头尾/带PEM一行/带PEM多行；SHA256 先 SHA1 后）
+  if (publicKey) {
+    let pem = String(publicKey)
+    if (!pem.includes('-----BEGIN')) {
+      pem = `-----BEGIN PUBLIC KEY-----\n${pem}\n-----END PUBLIC KEY-----`
+    } else if (!pem.includes('\n')) {
+      pem = pem
+        .replace('-----BEGIN PUBLIC KEY-----', '-----BEGIN PUBLIC KEY-----\n')
+        .replace('-----END PUBLIC KEY-----', '\n-----END PUBLIC KEY-----')
+    }
+    for (const alg of ['RSA-SHA256', 'RSA-SHA1']) {
+      try {
+        const verify = crypto.createVerify(alg)
+        verify.update(signStr, 'utf8')
+        if (verify.verify(pem, sign, 'base64')) return true
+      } catch (e) {}
+    }
+  }
+  // 兜底：通联官方生产平台公钥（1024-bit）+ RSA-SHA1。
+  // 这是因为 union_publicKey 配置通常是商户自己生成的应用公钥，用于商户端请求签名的「请求方向」，
+  // 而通联平台异步通知的「回调方向」签名用的是通联自己掌握的另一对密钥（老平台是 1024-bit SHA1）。
+  if (
+    fallbackPlatformPublicKey &&
+    (!publicKey || String(publicKey) !== String(fallbackPlatformPublicKey))
+  ) {
+    try {
+      const pem = `-----BEGIN PUBLIC KEY-----\n${fallbackPlatformPublicKey}\n-----END PUBLIC KEY-----`
+      const verify = crypto.createVerify('RSA-SHA1')
+      verify.update(signStr, 'utf8')
+      if (verify.verify(pem, sign, 'base64')) return true
+    } catch (e) {}
   }
+  return false
 }
 
+// 通联支付官方生产环境 1024-bit 平台公钥（用于回调验签兜底，与 allinpay.js 内置 publicKey.production 保持一致）
+// 商户的 union_publicKey 配置通常是自己生成的 2048-bit 应用公钥（给商户侧请求签名用），和通联平台用于
+// 异步通知签名的私钥不是同一对，所以当 union_publicKey 验签失败时，再用这张通联官方平台公钥补一轮。
+const UNION_PROD_PLATFORM_PUBLIC_KEY =
+  'MIGfMA0GCSqGSIb3DQEBAQUAA4GNADCBiQKBgQCm9OV6zH5DYH/ZnAVYHscEELdCNfNTHGuBv1nYYEY9FrOzE0/4kLl9f7Y9dkWHlc2ocDwbrFSm0Vqz0q2rJPxXUYBCQl5yW3jzuKSXif7q1yOwkFVtJXvuhf5WRy+1X5FOFoMvS7538No0RpnLzmNi3ktmiqmhpcY/1pmt20FHQQIDAQAB'
+
 const check = ({ timestamp, nonce, signature, token }) => {
   const tmp = [token, timestamp, nonce].sort().join('')
   const currSign = crypto.createHash('sha1').update(tmp).digest('hex')
@@ -1569,6 +1604,7 @@ exports.union_notify = async (ctx) => {
         signType: rawSignType,
         publicKey: union_publicKey,
         md5Key: union_md5Key,
+        fallbackPlatformPublicKey: UNION_PROD_PLATFORM_PUBLIC_KEY,
       })
       if (!signOk) {
         // ===== 诊断信息：帮助定位到底是 publicKey 错、签名串错、还是配置根本没拿到值 =====
@@ -1581,22 +1617,8 @@ exports.union_notify = async (ctx) => {
           app,
           message: `通联回调验签失败【详细诊断】type=${rawSignType} cusid=${cusid} appid=${notifyData.appid} trxid=${notifyData.trxid} | union_publicKey=${keyTypeInfo} | union_publicKey_isSet=!!${!!union_publicKey} | union_md5Key_isSet=!!${!!union_md5Key} | signStr_len=${diagsignStr.length} | signStr=${diagsignStr} | raw_data=${JSON.stringify(notifyData)}`,
         })
-        // TODO: 2026-09-02 临时绕过（上线确认验签算法+公钥正确后删除）：
-        //   只有当「商户号绑定正确」+「trxstatus=0000」+「存在 sign」时才允许绕过，
-        //   避免无 sign 的伪造报文直接进业务。
-        if (
-          trxstatus === '0000' &&
-          union_mchId &&
-          cusid === union_mchId
-        ) {
-          app.service.log.push({
-            app,
-            message: `通联回调验签临时BYPASS: trxid=${notifyData.trxid} out_trade_no=${notifyData.cusorderid} (24h内确认后删除此逻辑)`,
-          })
-        } else {
-          ctx.body = 'FAIL'
-          return
-        }
+        ctx.body = 'FAIL'
+        return
       }
     }
 

package/core/file/services/allinpay.js

@@ -67,10 +67,15 @@ class AllinPay {
     const signStr = this.buildSignStr(params)
 
     if (this.signType === 'RSA') {
-      // 兼容三种配置格式：已经带 PEM 头尾直接使用；否则默认包 PKCS#1 (BEGIN RSA PRIVATE KEY) 头尾
+      // PEM 标准化：已有头尾直接用（缺换行补换行）；否则默认包 PKCS#1 (BEGIN RSA PRIVATE KEY) 头尾
       let pem
-      if (String(this.privateKey).includes('-----BEGIN')) {
-        pem = String(this.privateKey)
+      const raw = String(this.privateKey)
+      if (raw.includes('-----BEGIN')) {
+        pem = raw.includes('\n')
+          ? raw
+          : raw
+              .replace(/-----(BEGIN[^-]+KEY)-----/, '-----$1-----\n')
+              .replace(/-----(END[^-]+KEY)-----/, '\n-----$1-----')
       } else {
         pem = `-----BEGIN RSA PRIVATE KEY-----\n${this.privateKey}\n-----END RSA PRIVATE KEY-----`
       }
@@ -99,15 +104,24 @@ class AllinPay {
 
     const signStr = this.buildSignStr(paramsCopy)
 
-    // 兼容两种配置格式：已经带 PEM 头尾的直接用；纯 base64 body 再包裹头尾
-    const pem = String(this.publicKey).includes('-----BEGIN')
-      ? String(this.publicKey)
-      : `-----BEGIN PUBLIC KEY-----\n${this.publicKey}\n-----END PUBLIC KEY-----`
-    const verify = crypto.createVerify('RSA-SHA1')
-    verify.update(signStr, 'utf8')
-    const result = verify.verify(pem, sign, 'base64')
-
-    return result
+    // PEM 标准化：无头尾则包 SPKI 头尾 + 内容单换行；已有头尾若一行无换行补换行
+    let pem = String(this.publicKey)
+    if (!pem.includes('-----BEGIN')) {
+      pem = `-----BEGIN PUBLIC KEY-----\n${pem}\n-----END PUBLIC KEY-----`
+    } else if (!pem.includes('\n')) {
+      pem = pem
+        .replace('-----BEGIN PUBLIC KEY-----', '-----BEGIN PUBLIC KEY-----\n')
+        .replace('-----END PUBLIC KEY-----', '\n-----END PUBLIC KEY-----')
+    }
+    // RSA 双算法兜底：先 SHA256（新 2048-bit 证书），失败再 SHA1（老 1024-bit 证书）
+    for (const alg of ['RSA-SHA256', 'RSA-SHA1']) {
+      try {
+        const verify = crypto.createVerify(alg)
+        verify.update(signStr, 'utf8')
+        if (verify.verify(pem, sign, 'base64')) return true
+      } catch (e) {}
+    }
+    return false
   }
 
   async request(url, params) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "q-koa",
-  "version": "13.8.37",
+  "version": "13.8.39",
   "description": "",
   "main": "index.js",
   "scripts": {