Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent Sovrn TypeError #664

Merged
merged 1 commit into from
Sep 30, 2016
Merged

Prevent Sovrn TypeError #664

merged 1 commit into from
Sep 30, 2016

Conversation

matthewlane
Copy link
Collaborator

Type of change

  • Bugfix

Description of change

If find returns undefined, attempting to access properties of missing will throw a TypeError. This occasionally causes uncaught exceptions when using Karma in debug mode (gulp serve --watch, check in dev tools console). Fixing this may help stabilize the unit test failures observed in #642.

Other information

@prebid/sovrn, please review this change or feel free to submit a PR that handles this exception.

If `find` returns `undefined`, attempting to access properties of `missing` will throw a TypeError.
Copy link
Collaborator

@protonate protonate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@protonate protonate merged commit 4b150bf into master Sep 30, 2016
@protonate protonate deleted the prevent-sovrn-TypeError branch September 30, 2016 18:01
Studnicky pushed a commit to sonobi/Prebid.js that referenced this pull request Oct 4, 2016
If `find` returns `undefined`, attempting to access properties of `missing` will throw a TypeError.
@matthewlane matthewlane mentioned this pull request Oct 17, 2016
2 tasks
marian-r pushed a commit to aol/Prebid.js that referenced this pull request Nov 2, 2016
…ebid-0.13.1 to release/1.4.0

* commit 'd174ef271e55e7f26210f50caac52c0a036bc9a1':
  CHANGELOG.
  0.13.1 release
  add an --https flag to run local dev server over https (prebid#670)
  test ie variant of url parse pathname (prebid#669)
  Prevent SpringServe TypeError (prebid#663)
  XDomainRequest does not support `readyState` and was not executing the callback. (prebid#668)
  Prevent Sovrn TypeError (prebid#664)
  Prevent TripleLift TypeError (prebid#662)
  fixed bug with using non standard "standard" keys and sendAllBids (prebid#665)
  Initial value 0 for adder. (prebid#656)
  Prevent bidmanager TypeError (prebid#661)
  Prevent Pubmatic TypeError (prebid#666)
  Size mapping functionality (prebid#651)
  Added '320x80': 59, '320x320': 72, '320x160': 73 to RUBICON_SIZE_MAP (prebid#649)
  Add defy alias + increment version. (prebid#650)
  Add additional sizes to rubicon size mapping (prebid#646)
marian-r added a commit to aol/Prebid.js that referenced this pull request Nov 2, 2016
…4.0 to master

* commit '5e7eec42d84ec5de47a28ba493a9357b595761d3':
  CHANGELOG.
  New adapter.
  0.13.1 release
  add an --https flag to run local dev server over https (prebid#670)
  test ie variant of url parse pathname (prebid#669)
  Prevent SpringServe TypeError (prebid#663)
  XDomainRequest does not support `readyState` and was not executing the callback. (prebid#668)
  Prevent Sovrn TypeError (prebid#664)
  Prevent TripleLift TypeError (prebid#662)
  fixed bug with using non standard "standard" keys and sendAllBids (prebid#665)
  Initial value 0 for adder. (prebid#656)
  Prevent bidmanager TypeError (prebid#661)
  Prevent Pubmatic TypeError (prebid#666)
  Size mapping functionality (prebid#651)
  Added '320x80': 59, '320x320': 72, '320x160': 73 to RUBICON_SIZE_MAP (prebid#649)
  Add defy alias + increment version. (prebid#650)
  Add additional sizes to rubicon size mapping (prebid#646)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants