MailOnline / videojs-vast-vpaid

video.js vast plugin
MIT License
296 stars 231 forks source link

VAST 3 and ad pods #198

Open fana605 opened 8 years ago

fana605 commented 8 years ago

Hello, Will the plugin work with VAST 3 format? I'm triying the demo at http://mailonline.github.io/videojs-vast-vpaid/index_4.html and I'm getting this error on the console: AD ERROR: VAST Error: on VASTClient.getVASTAd.buildAd, error parsing xml VASTError {message: "VAST Error: on VASTClient.getVASTAd.buildAd, error parsing xml", code: 100} []

The xml is this: http://www8.smartadserver.com/ac?siteid=99082&pgid=645173&fmtid=40646&ab=1&tgt=&oc=1&out=vast&ps=2&pb=0&visit=S&vcn=s&tmstp=4653465465

Thank you

Fetz commented 8 years ago

Hi yes is works with most of the cases. Is not working with that one because there is an Error with parser not handling empty AdParameters.

<AdParameters xmlEncoded="true">
<![CDATA[ ]]>
</AdParameters>

I will try to fix it tomorrow