fix: Be defensive when iteration over object properties

[kamilogorek]

Fixes #1168

[codecov-io]

Codecov Report

Merging #1521 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1521   +/-   ##
=======================================
  Coverage   73.69%   73.69%           
=======================================
  Files          45       45           
  Lines        1361     1361           
  Branches      257      257           
=======================================
  Hits         1003     1003           
  Misses        350      350           
  Partials        8        8

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ea3b09d...f90128d. Read the comment docs.

[atk]

This still breaks integration when an external module iterates over the properties of the console methods or other wrapped functions. I would suggest adding the additional properties as not enumerable, i.e.

Object.defineProperties(wrapped, {
  __sentry__: { value: true, enumerable: false },
  __sentry_original__: { value: original, enumerable: false },
  __sentry_wrapped__: { value: wrapped, enumerable: false }
});

instead of

wrapped.__sentry__ = true;
wrapped.__sentry_original__ = original;
wrapped.__sentry_wrapped__ = wrapped;

[kamilogorek]

@atk you're absolutely right. Created an issue for that – #1849