truncateDeep() throws with objects not inheriting hasOwnProperty()

[brianreavis]

bugsnag-js/src/bugsnag.js

Line 761 in 67753db

if (object.hasOwnProperty(key)) {

This line is not safe and breaks history.pushState and history.replaceState when state contains object(s) that do not inherit hasOwnProperty (e.g. were created with Object.create(null)).

Observed behavior

TypeError: e.hasOwnProperty is not a function

Steps to reproduce

• Have auto breadcrumbs enabled and call history.pushState with a state containing object(s) that do not not inherit hasOwnProperty.

var plainObject = Object.create(null);
plainObject.dummyProperty = true;
var state = {myObject: plainObject};
history.pushState(state, 'foo', 'bar.html');

Version

3.x

[bengourley]

Hey @brianreavis, thanks for the bug report. I'll get this looked into.

[brianreavis]

Thanks @bengourley!

[bengourley]

Hey @brianreavis, v3.3.1 was just released containing the fix for this. If you are using the CDN link, the latest version will be fetched automatically and should just work™. Otherwise, please manually upgrade and let me know how you get on. Cheers!