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

Getting an error "SyntaxError: Document.querySelectorAll: 'svrl:failed-assert' is not a valid selector" for any MPD stream, even the samples from DASH-IF site. #713

Open
fkerrin opened this issue Nov 8, 2024 · 8 comments

Comments

@fkerrin
Copy link

fkerrin commented Nov 8, 2024

Please see the attached screenshot
DASH-error

@rbouqueau
Copy link
Collaborator

Thank you. I get the error on staging too. Please allow me to restart and retry in a few minutes.

@rbouqueau
Copy link
Collaborator

Still failing after reboot. @Phencys ?

@guan1
Copy link

guan1 commented Nov 8, 2024

Hi, I also tried it and I'm still seeing this error.

Best regards

@Phencys
Copy link
Collaborator

Phencys commented Nov 12, 2024

I've been able to reproduce this issue in Firefox, strangely enough Chrome seems to accept invalid XML without any form of notification.

The mpdvalidator outputs a namespaced XML, but does not include the namespace in its output. XML allows the semicolon : only in case of valid namespacing, and Firefox enforces this correctly.

I have prepared a fix by replacing the : by an underscore creating a valid xml again #714 . This fix is now available on Staging now.

@fkerrin @guan1 can either of you confirm the message doesn't show up anymore?

@guan1
Copy link

guan1 commented Nov 12, 2024

works for me now. Thanks.

@fkerrin
Copy link
Author

fkerrin commented Nov 12, 2024

Hi @Phencys. I don't have access to staging, I'm just Joe Public and for me the problem still persists. Regards,

@Phencys
Copy link
Collaborator

Phencys commented Nov 12, 2024

Hello @fkerrin

Staging is a public hosted environment which we use to pre-deploy fixes :)

It is available on the link in the previous message, or if you prefer to copy-paste on https://staging.conformance.dashif.org/

@Phencys Phencys reopened this Nov 12, 2024
@fkerrin
Copy link
Author

fkerrin commented Nov 12, 2024

Thank you @Phencys - that worked for me. Regards.

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

No branches or pull requests

4 participants