resolve with warning rather than reject if client is undefined

Author: lobsterkatie

packages/browser/src/sdk.ts

@@ -156,7 +156,8 @@ export function flush(timeout?: number): PromiseLike<boolean> {
   if (client) {
     return client.flush(timeout);
   }
-  return SyncPromise.reject(false);
+  logger.warn('Cannot flush events. No client defined.');
+  return SyncPromise.resolve(false);
 }
 
 /**
@@ -170,7 +171,8 @@ export function close(timeout?: number): PromiseLike<boolean> {
   if (client) {
     return client.close(timeout);
   }
-  return SyncPromise.reject(false);
+  logger.warn('Cannot flush events and disable SDK. No client defined.');
+  return SyncPromise.resolve(false);
 }
 
 /**

packages/node/src/sdk.ts

@@ -1,7 +1,7 @@
 import { getCurrentHub, initAndBind, Integrations as CoreIntegrations } from '@sentry/core';
 import { getMainCarrier, setHubOnCarrier } from '@sentry/hub';
 import { SessionStatus } from '@sentry/types';
-import { getGlobalObject } from '@sentry/utils';
+import { getGlobalObject, logger } from '@sentry/utils';
 import * as domain from 'domain';
 
 import { NodeClient } from './client';
@@ -150,7 +150,8 @@ export async function flush(timeout?: number): Promise<boolean> {
   if (client) {
     return client.flush(timeout);
   }
-  return Promise.reject(false);
+  logger.warn('Cannot flush events. No client defined.');
+  return Promise.resolve(false);
 }
 
 /**
@@ -164,7 +165,8 @@ export async function close(timeout?: number): Promise<boolean> {
   if (client) {
     return client.close(timeout);
   }
-  return Promise.reject(false);
+  logger.warn('Cannot flush events and disable SDK. No client defined.');
+  return Promise.resolve(false);
 }
 
 /**