fix(jira): use Accept: */* when downloading attachment content (#577)

The Jira attachment-content endpoint
(`/rest/api/3/attachment/content/{id}`) serves the file's own media type and
responds 406 Not Acceptable to a narrow `Accept: application/octet-stream`.
That made every attachment download fail with HTTP 406, and the fail-closed
path then rejected the whole task ("could not be downloaded (HTTP 406)").
Send `Accept: */*` so the gateway serves the real content type.

Verified against the live endpoint: octet-stream → 406, */* → 200.
Test asserts the download request sets Accept: */*.

Author: ayushtr-aws

cdk/src/handlers/shared/jira-attachments.ts

@@ -264,7 +264,11 @@ async function fetchAttachmentBytes(
       method: 'GET',
       headers: {
         Authorization: `Bearer ${accessToken}`,
-        Accept: 'application/octet-stream',
+        // The attachment-content endpoint returns the file's own content type
+        // (image/jpeg, application/pdf, …) and responds 406 Not Acceptable to a
+        // narrow `Accept: application/octet-stream`. Accept anything so the
+        // gateway can serve the real media type.
+        Accept: '*/*',
       },
       signal: controller.signal,
     });

cdk/test/handlers/shared/jira-attachments.test.ts

@@ -141,6 +141,10 @@ describe('downloadScreenAndStoreJiraAttachments', () => {
     // GET was addressed by attachment id on the gateway base, not the raw content URL.
     const fetchUrl = (global.fetch as jest.Mock).mock.calls[0][0] as string;
     expect(fetchUrl).toContain('api.atlassian.com/ex/jira/cloud-1/rest/api/3/attachment/content/att-1');
+    // Accept must be permissive: the content endpoint 406s on
+    // `application/octet-stream` and serves the file's own media type.
+    const fetchInit = (global.fetch as jest.Mock).mock.calls[0][1] as { headers: Record<string, string> };
+    expect(fetchInit.headers.Accept).toBe('*/*');
   });
 
   test('routes images through screenImage and marks type image', async () => {