fix: Normalize data before passing it to transport

Author: kamilogorek

CHANGELOG.md

@@ -5,12 +5,15 @@
 - [core] feat: Deprecate `captureEvent`, prefer `sendEvent` for transports. `sendEvent` now takes a string (body)
   instead of `Event` object.
 - [core] feat: Use correct buffer for requests in transports
-- [node] feat: Add file cache for providing pre/post context in frames
-- [node] feat: New option `frameContextLines`, if set to `0` we do not provide source code pre/post context, default is
-  `7` lines pre/post
 - [core]: ref: Change way how transports are initialized
 - [core]: ref: Rename `RequestBuffer` to `PromiseBuffer`, also introduce limit
+- [core]: fix: Check if value is error object in extraErrorData integration
 - [browser] fix: Prevent empty exception values
+- [browser]: fix: Permission denied to access property name
+- [node] feat: Add file cache for providing pre/post context in frames
+- [node] feat: New option `frameContextLines`, if set to `0` we do not provide source code pre/post context, default is
+  `7` lines pre/post
+- [utils] fix: Use custom serializer inside `serialize` method to prevent circular references
 
 ## 4.4.2
 

packages/browser/src/integrations/trycatch.ts

@@ -210,9 +210,12 @@ export class TryCatch implements Integration {
   }
 }
 
+/**
+ * Safely extract function name from itself
+ */
 function getFunctionName(fn: any): string {
   try {
-    return fn && fn.name || '<anonymous>';
+    return (fn && fn.name) || '<anonymous>';
   } catch (e) {
     // Just accessing custom props in some Selenium environments
     // can cause a "Permission denied" exception (see raven-js#495).

packages/utils/src/object.ts

@@ -11,17 +11,13 @@ interface ExtendedError extends Error {
 /**
  * Serializes the given object into a string.
  * Like JSON.stringify, but doesn't throw on circular references.
- * Based on a `json-stringify-safe` package and modified to handle Errors serialization.
- *
- * The object must be serializable, i.e.:
- *  - Only primitive types are allowed (object, array, number, string, boolean)
- *  - Its depth should be considerably low for performance reasons
+ * and modified to handle Errors and "unserializable values" serialization.
  *
  * @param object A JSON-serializable object.
  * @returns A string containing the serialized object.
  */
 export function serialize<T>(object: T): string {
-  return JSON.stringify(object);
+  return JSON.stringify(object, standardizer());
 }
 
 /**