Player.js: TypeError: undefined is not an object (evaluating 't.data.method')

[joker-777]

Actual Behavior

Without any changes from our site we suddenly can track that our users receive this error TypeError: undefined is not an object (evaluating 't.data.method') which is caused in this line.

Steps to Reproduce

We can't really reproduce it but we see in our js error tracking system that lots of people are having this error. Have you guys changed anything recently which could be the reason for this error?
We also updated @vimeo/player but it didn't change anything.

[rainerborene]

I can confirm this issue but the error message is a little bit different than yours: TypeError: Cannot read property 'method' of undefined

[rainerborene]

can't reproduce it either 😢

[joker-777]

@rainerborene Yes, we have both error message. It depends on the browser/devise.

[danaburger]

links where described issue occurs-

http://jsfiddle.net/joker777/sLyco0az/2/
https://www.kenhub.com/en/videos/upper-arm-bones-and-ligaments

Note: issue is not reproducible but does come up in logs.

[joker-777]

@danaburger Thanks for adding our details. Here some stats about this error from our side.

Please be aware that we sample the errors we get by 20%. That means we actually only get 20% of all error occurrences. The real numbers are definitely much higher.

[ilyen85]

Same error happens on our page also at the very same line, with the following error message:

TypeError: Cannot read property 'method' of undefined

[joaovieira]

Still happening very often.. is this being looked at? I assume it's an error message from the player iframe, so there's no proper stack trace.

Could we just safe-guard in the processData to check for undefined data before accessing the method?

[reckoner165]

@joker-777 @joaovieira @rainerborene We are working to resolve this. At the moment, postMessages with event == 'error' are not fired by the API, which will be fixed in the next release.
Meanwhile, can you provide a stack trace of the sentry log for more context around the issue? Thanks!

[danaburger]

This issue should now be resolved. @joker-777 @joaovieira @rainerborene @ilyen85 are you still seeing errors?

[joker-777]

@danaburger The last seen error was at Sep 7, 2018 10:34:37 AM UTC but this could be also related to caching. Let's have a look how it will continue.

May I ask how you resolved this? What was exactly the issue?

[joker-777]

@danaburger It doesn't look like that the error is fixed. Do I have to update something?

[danaburger]

@joker-777 @joaovieira Sorry to hear the issue is still occurring. To help us get to the bottom of this, could you provide a stack trace around your sentry error logs?

[rainerborene]

@danaburger I don't see any change pushed to the master branch.

[danaburger]

@rainerborene The push was recorded internally, as we had to make an adjustment our player and not the API.

If you are still seeing these errors, would you mind providing a stack trace around your error logs? These should help us move forward.

[rainerborene]

Just received another notification on Sentry a few minutes ago. It's not fixed yet.

For now, I'll mute this annoying error directly on Sentry.

[joker-777]

Hi, here is the stack trace although I don't think it is so much helpful

TypeError: Unable to get property 'method' of undefined or null reference
  at processData(application.js:75059:7)
  at o(application.js:75167:9)

TypeError: t.data is undefined
  at processData(application.js:75059:11)
  at func(application.js:75174:9)

As I already mentioned in the description, the error occurs here

https://github.com/vimeo/player.js/blob/d953eadc5a82ce46ff48b984e1688210c40f08bd/src/lib/postmessage.js#L65

As far as I can see, the player listens to message events and receives error events which don't have any data payload.

https://github.com/vimeo/player.js/blob/adefaad4d03ec8135553d21c65cf09f12c20ffea/src/player.js#L85

[micahsp]

I am seeing similar errors starting 08/07/2018 and still present. Only on iOS.

TypeError: undefined is not an object (evaluating 'o.data.method')
File https://player.vimeo.com/api/player.js line 2 col 8633 in b
File https://player.vimeo.com/api/player.js line 2 col 10186 in e

[luwes]

This is resolved.

Quick test is to go to https://player.vimeo.com/api/demo and click the random color button a few times, it will throw an error eventually which is now handled properly instead of the error explained above.

[joker-777]

@luwes @danaburger The error is back. Our js error tracking system is tracking quite a lot of them.

[joaovieira]

Same.

[rainerborene]

Same here.

On Thu, Oct 11, 2018, 06:36 João Vieira [e-mail] wrote:

Same.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vimeo/player.js/issues/308#issuecomment-428888950,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGMfSAg5xHgKBs9JbbBg4r_bl5BUrBMks5ujxE1gaJpZM4WJgD7
.

[jinsoocha]

Same here

[hootlex]

Same.

[DarosF]

same

[mouthzipper]

same issue

[Qcasares]

Same issue