bitmovin / bitmovin-player-android-analytics-conviva

Integration of the Bitmovin Android Player SDK with the Conviva Analytics
MIT License
1 stars 4 forks source link

Feat/Stack trace #53

Open Wkkkkk opened 1 year ago

Wkkkkk commented 1 year ago

This commit addresses XYEYE- 12 (Android stack trace feature) and is based on the latest Conviva SDK.

While waiting for the Conviva validation report, we tried to retrieve the stack trace of Player's/Source's Error events and report it to TouchStone. It was tested with a sample stream with ads. In case of errors, a custom event will be fired containing stack trace information, which can be observed in TouchStone.

In the ideal case (no issues from the validation report), this PR will also complete XYEYE- 15 (Android update).

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

CLAassistant commented 1 year ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Wkkkkk commented 1 year ago

I've signed that. Let's wait for the validation.

dweinber commented 1 year ago

Something looks a bit fishy here: image

Anyway, that's something for us to figure out, no need to worry for you. As at least one of the two messages shows up as signed, that should be fine - thanks! 👍

Cool, looking forward to Conviva validation feedback, let's hope the best 🤞

kannan-sivam commented 1 week ago

The PR validation was successfully completed by team conviva. Error reporting for both Video Start Failure and Video Playback Failure is functioning as expected, displaying the correct error codes and messages.