fix(storage): set application/json Content-Type for impersonated ADC requests (#8419)

* feat(storage): set Content-Type to application/json in decorateRequest when json is present and header is missing

* fix(storage): ensure Headers instance supports 'has' method before setting content-type and skip test if Headers is undefined

* fix: mutating issue

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

---------

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

Author: thiyaguk09

handwritten/storage/src/nodejs-common/util.ts

@@ -970,6 +970,24 @@ export class Util {
       delete reqOpts.json.autoPaginate;
       delete reqOpts.json.autoPaginateVal;
       reqOpts.json = replaceProjectIdToken(reqOpts.json, projectId);
+
+      const headers = reqOpts.headers || {};
+      if (
+        typeof (headers as any).set === 'function' &&
+        typeof (headers as any).has === 'function'
+      ) {
+        if (!(headers as any).has('content-type')) {
+          (headers as any).set('Content-Type', 'application/json');
+        }
+        reqOpts.headers = headers;
+      } else {
+        const hasContentType = Object.keys(headers).some(
+          key => key.toLowerCase() === 'content-type'
+        );
+        reqOpts.headers = hasContentType
+          ? headers
+          : { ...headers, 'Content-Type': 'application/json' };
+      }
     }
 
     reqOpts.uri = replaceProjectIdToken(reqOpts.uri, projectId);

handwritten/storage/test/nodejs-common/util.ts

@@ -1805,6 +1805,64 @@ describe('common/util', () => {
       assert.deepStrictEqual(decoratedRequest.json, decoratedJson);
     });
 
+    it('should set Content-Type header on plain headers object when json is set', () => {
+      const projectId = 'project-id';
+      const reqOpts = {
+        uri: 'http://',
+        json: {},
+        headers: {},
+      };
+      replaceProjectIdTokenOverride = (x: any) => x;
+
+      const decoratedRequest = util.decorateRequest(reqOpts, projectId);
+      assert.strictEqual(
+        (decoratedRequest.headers as any)['Content-Type'],
+        'application/json'
+      );
+    });
+
+    it('should set Content-Type header on Headers instance when json is set', () => {
+      if (typeof Headers === 'undefined') {
+        return;
+      }
+      const projectId = 'project-id';
+      const headersInstance = new Headers();
+      const reqOpts = {
+        uri: 'http://',
+        json: {},
+        headers: headersInstance,
+      };
+      replaceProjectIdTokenOverride = (x: any) => x;
+
+      const decoratedRequest = util.decorateRequest(reqOpts as any, projectId);
+      assert.strictEqual(
+        (decoratedRequest.headers as any).get('Content-Type'),
+        'application/json'
+      );
+    });
+
+    it('should not overwrite existing Content-Type header if already present', () => {
+      const projectId = 'project-id';
+      const reqOpts = {
+        uri: 'http://',
+        json: {},
+        headers: {
+          'content-type': 'application/x-protobuf',
+        },
+      };
+      replaceProjectIdTokenOverride = (x: any) => x;
+
+      const decoratedRequest = util.decorateRequest(reqOpts, projectId);
+      assert.strictEqual(
+        (decoratedRequest.headers as any)['content-type'],
+        'application/x-protobuf'
+      );
+      assert.strictEqual(
+        (decoratedRequest.headers as any)['Content-Type'],
+        undefined
+      );
+    });
+
     it('should decorate the request', () => {
       const projectId = 'project-id';
       const reqOpts = {