Added logic that stringifies releases if release is an object in browser/node

Author: ahmedetefy

packages/browser/src/sdk.ts

@@ -83,7 +83,10 @@ export function init(options: BrowserOptions = {}): void {
     if (window.SENTRY_RELEASE && window.SENTRY_RELEASE.id) {
       options.release = window.SENTRY_RELEASE.id;
     }
+  } else if (typeof options.release !== 'string') {
+    options.release = JSON.stringify(options.release);
   }
+
   if (options.autoSessionTracking === undefined) {
     options.autoSessionTracking = true;
   }

packages/browser/test/unit/index.test.ts

@@ -185,6 +185,13 @@ describe('SentryBrowser initialization', () => {
     expect(global.__SENTRY__.hub._stack[0].client.getOptions().release).to.be.undefined;
   });
 
+  it('should use window.SENTRY_RELEASE to set release on initialization if available', () => {
+    const releaseObj = { name: 'release hello world' };
+    // @ts-ignore
+    init({ dsn, release: releaseObj });
+    expect(global.__SENTRY__.hub._stack[0].client.getOptions().release).equal(JSON.stringify(releaseObj));
+  });
+
   describe('SDK metadata', () => {
     it('should set SDK data when Sentry.init() is called', () => {
       init({ dsn });

packages/node/src/sdk.ts

@@ -97,6 +97,8 @@ export function init(options: NodeOptions = {}): void {
     if (detectedRelease !== undefined) {
       options.release = detectedRelease;
     }
+  } else if (typeof options.release !== 'string') {
+    options.release = JSON.stringify(options.release);
   }
 
   if (options.environment === undefined && process.env.SENTRY_ENVIRONMENT) {

packages/node/test/index.test.ts

@@ -279,6 +279,13 @@ describe('SentryNode initialization', () => {
     global.SENTRY_RELEASE = undefined;
   });
 
+  test('release is stringified, if options.release is passed as an object', () => {
+    const releaseObj = { name: 'release hello world' };
+    // @ts-ignore
+    init({ dsn, release: releaseObj });
+    expect(global.__SENTRY__.hub._stack[0].client.getOptions().release).toEqual(JSON.stringify(releaseObj));
+  });
+
   describe('SDK metadata', () => {
     it('should set SDK data when Sentry.init() is called', () => {
       init({ dsn });