q-koa 13.8.38 → 13.8.40

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)
@@ -121,28 +127,47 @@ const validateUnionNotifySign = ({ notifyData, signType, publicKey, md5Key }) =>
   // RSA 双算法兜底：先试 RSA-SHA256（通联 2048-bit 新平台证书），失败再退 RSA-SHA1（1024-bit 老平台证书）
   // 两份证书的签名/密钥长度天然不同，1024-bit 公钥永远验不出 256 字节签名，2048-bit 公钥也验不出 128 字节签名，
   // 所以顺序上即使「错算法试一次」也不会误判，只是多一次 CPU crypto 调用（微秒级，可忽略）。
-  if (!publicKey) return false
-  // PEM 标准化：无头尾则包 SPKI 头尾 + 内容单换行；已有头尾若一行无换行，补换行让 OpenSSL 稳定识别
-  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-----')
+  // 尝试用户配置的 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) {}
+    }
   }
-  for (const alg of ['RSA-SHA256', 'RSA-SHA1']) {
+  // 兜底：通联官方生产平台公钥（1024-bit）+ RSA-SHA1。
+  // 这是因为 union_publicKey 配置通常是商户自己生成的应用公钥，用于商户端请求签名的「请求方向」，
+  // 而通联平台异步通知的「回调方向」签名用的是通联自己掌握的另一对密钥（老平台是 1024-bit SHA1）。
+  if (
+    fallbackPlatformPublicKey &&
+    (!publicKey || String(publicKey) !== String(fallbackPlatformPublicKey))
+  ) {
     try {
-      const verify = crypto.createVerify(alg)
+      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) {
-      // key 跟算法不兼容（比如 1024-bit 公钥试 SHA256 在极旧 Node 下可能 throw），静默下一轮
-    }
+    } 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')
@@ -1514,12 +1539,6 @@ exports.union_notify = async (ctx) => {
   try {
     const notifyData = ctx.request.body
 
-    // TODO: 临时观察日志，上线后删除
-    app.service.log.push({
-      app,
-      message: `通联融合回调union_notify报文: ${JSON.stringify(notifyData || {})}`,
-    })
-
     if (!notifyData || typeof notifyData !== 'object') {
       ctx.status = 400
       ctx.body = 'FAIL'
@@ -1563,50 +1582,20 @@ exports.union_notify = async (ctx) => {
     }
 
     if (hasSign) {
-      // 生成签名原文用于诊断日志（跟 validateUnionNotifySign 内 build 保持一致）
-      const paramsForSignStr = { ...notifyData }
-      delete paramsForSignStr.sign
-      const sortedKeysForSign = Object.keys(paramsForSignStr).sort()
-      let diagsignStr = ''
-      for (const k of sortedKeysForSign) {
-        const v = paramsForSignStr[k]
-        if (v === undefined || v === null || v === '') continue
-        if (diagsignStr) diagsignStr += '&'
-        diagsignStr += `${k}=${v}`
-      }
       const signOk = validateUnionNotifySign({
         notifyData,
         signType: rawSignType,
         publicKey: union_publicKey,
         md5Key: union_md5Key,
+        fallbackPlatformPublicKey: UNION_PROD_PLATFORM_PUBLIC_KEY,
       })
       if (!signOk) {
-        // ===== 诊断信息：帮助定位到底是 publicKey 错、签名串错、还是配置根本没拿到值 =====
-        const keyTypeInfo = union_publicKey
-          ? (String(union_publicKey).includes('-----BEGIN')
-              ? `pem-with-head,len=${String(union_publicKey).length},head=${String(union_publicKey).slice(0, 24)}`
-              : `pem-body-only,len=${String(union_publicKey).length},head=${String(union_publicKey).slice(0, 24)}`)
-          : 'NULL/UNDEFINED'
         app.service.log.push({
           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)}`,
+          message: `通联回调验签失败(type=${rawSignType}): ${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
       }
     }
 
@@ -1703,11 +1692,6 @@ exports.union_notify = async (ctx) => {
 exports.notify = async (ctx) => {
   const { app } = getAppByCtx(ctx)
   const result = ctx.request.xmlBody
-  // TODO: 临时观察日志，上线后删除
-  app.service.log.push({
-    app,
-    message: `微信支付回调notify报文: ${JSON.stringify(result || {})}`,
-  })
   if (!result || !result.xml) return sendWxFail(ctx, 'INVALID_BODY')
 
   // ===== 微信 V2 回调验签 + mch_id 匹配 =====
@@ -1982,11 +1966,6 @@ const getRefundResultJson = async ({ req_info, app_key }) => {
 exports.refund_notify = async (ctx) => {
   const { app } = getAppByCtx(ctx)
   const result = ctx.request.xmlBody
-  // TODO: 临时观察日志，上线后删除
-  app.service.log.push({
-    app,
-    message: `微信退款回调refund_notify报文: ${JSON.stringify(result || {})}`,
-  })
   if (!result || !result.xml) return sendWxFail(ctx, 'INVALID_BODY')
 
   const pay_config = ctx.params.sub || 'weixin_pay'

package/package.json

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