[BUG] fix headerParams

Author: tokuhirom

generator/src/main/resources/line-bot-sdk-nodejs-generator/apiBody/normal.pebble

@@ -18,7 +18,8 @@
             "{{param.paramName}}": {{param.paramName}},
         {% endfor %}
         };
-    {% elseif op.hasHeaderParams %}const headerParams = {
+    {% endif %}
+    {% if op.hasHeaderParams %}const headerParams = {
         {% for param in op.headerParams -%}
             "{{param.baseName}}": {{param.paramName}},
         {% endfor %}
@@ -31,10 +32,10 @@
                 .replace("{{ "{" + param.paramName + "}" }}", String({{ param.paramName }}))
         {% endfor %},
 
-            {% if op.hasBodyParam %}params,{% endif %}
-            {% if op.hasFormParams %}formParams,{% endif %}
-            {% if op.hasQueryParams %}queryParams,{% endif %}
-            {% if op.hasHeaderParams %}headerParams,{% endif %}
+            {% if op.hasBodyParam %}params,
+            {% elseif op.hasFormParams %}formParams,
+            {% elseif op.hasQueryParams %}queryParams,{% endif %}
+            {% if op.hasHeaderParams %}{headers: headerParams},{% endif %}
         );
         return ensureJSON(res);
 {% endif %}

lib/messaging-api/api/messagingApiClient.ts

@@ -128,11 +128,15 @@ export class MessagingApiClient {
   ): Promise<object> {
     const params = broadcastRequest;
 
+    const headerParams = {
+      "X-Line-Retry-Key": xLineRetryKey,
+    };
+
     const res = this.httpClient.post<object>(
       "/v2/bot/message/broadcast",
       params,
 
-      headerParams,
+      { headers: headerParams },
     );
     return ensureJSON(res);
   }
@@ -779,11 +783,15 @@ export class MessagingApiClient {
   ): Promise<object> {
     const params = multicastRequest;
 
+    const headerParams = {
+      "X-Line-Retry-Key": xLineRetryKey,
+    };
+
     const res = this.httpClient.post<object>(
       "/v2/bot/message/multicast",
       params,
 
-      headerParams,
+      { headers: headerParams },
     );
     return ensureJSON(res);
   }
@@ -800,11 +808,15 @@ export class MessagingApiClient {
   ): Promise<object> {
     const params = narrowcastRequest;
 
+    const headerParams = {
+      "X-Line-Retry-Key": xLineRetryKey,
+    };
+
     const res = this.httpClient.post<object>(
       "/v2/bot/message/narrowcast",
       params,
 
-      headerParams,
+      { headers: headerParams },
     );
     return ensureJSON(res);
   }
@@ -821,11 +833,15 @@ export class MessagingApiClient {
   ): Promise<PushMessageResponse> {
     const params = pushMessageRequest;
 
+    const headerParams = {
+      "X-Line-Retry-Key": xLineRetryKey,
+    };
+
     const res = this.httpClient.post<PushMessageResponse>(
       "/v2/bot/message/push",
       params,
 
-      headerParams,
+      { headers: headerParams },
     );
     return ensureJSON(res);
   }
@@ -842,11 +858,15 @@ export class MessagingApiClient {
   ): Promise<Types.MessageAPIResponseBase> {
     const params = pnpMessagesRequest;
 
+    const headerParams = {
+      "X-Line-Delivery-Tag": xLineDeliveryTag,
+    };
+
     const res = this.httpClient.post(
       "/bot/pnp/push",
       params,
 
-      headerParams,
+      { headers: headerParams },
     );
     return ensureJSON(res);
   }

test/libs-messagingApi.spec.ts

@@ -37,4 +37,24 @@ describe("messagingApi", () => {
     equal(scope.isDone(), true);
     deepEqual(res, {});
   });
+
+  it("pushMessage", async () => {
+    const scope = nock("https://api.line.me/", {
+      reqheaders: {
+        Authorization: "Bearer test_channel_access_token",
+        "User-Agent": `${pkg.name}/${pkg.version}`,
+        "content-type": "application/json",
+        "x-line-retry-key": "KEYKEYKEYKEY",
+      },
+    })
+      .post("/v2/bot/message/push")
+      .reply(200, {});
+
+    const res = await client.pushMessage(
+      { to: "uAAAAAAAAAAAAAA", messages: [{ type: "text", text: "aaaaaa" }] },
+      "KEYKEYKEYKEY",
+    );
+    equal(scope.isDone(), true);
+    deepEqual(res, {});
+  });
 });