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

Parsing UTC timing response fails #147

Closed
dsilhavy opened this issue Jan 16, 2024 · 3 comments
Closed

Parsing UTC timing response fails #147

dsilhavy opened this issue Jan 16, 2024 · 3 comments

Comments

@dsilhavy
Copy link

The UTCTiming element in the livesim2 streams currently points to this URL: https://time.akamai.com/?ms. It uses a schemeIdUri set to urn:mpeg:dash:utc:http-xsdate:2014.

This combination leads to parsing errors in dash.js and the Shaka player. I think the correct URL for the schmeIdUri defined above is this one: https://time.akamai.com/?iso&ms

Example MPD: https://livesim2.dashif.org/livesim2/testpic_2s/Manifest.mpd

@tobbee
Copy link
Contributor

tobbee commented Jan 16, 2024

I thought the ISO in the URL meant that the format was iso format, so I have the table:

const (
	UtcTimingDirect       UTCTimingMethod = "direct"
	UtcTimingHead         UTCTimingMethod = "head"
	UtcTimingNtp          UTCTimingMethod = "ntp"
	UtcTimingSntp         UTCTimingMethod = "sntp"
	UtcTimingHttpXSDate   UTCTimingMethod = "httpxsdate"
	UtcTimingHttpXSDateMs UTCTimingMethod = "httpxsdatems"
	UtcTimingHttpISO      UTCTimingMethod = "httpiso"
	UtcTimingHttpISOMs    UTCTimingMethod = "httpisoms"
	UtcTimingNone         UTCTimingMethod = "none"
	UtcTimingKeep         UTCTimingMethod = "keep"
)

const (
	UtcTimingNtpServer          = "1.de.pool.ntp.org"
	UtcTimingSntpServer         = "time.kfki.hu"
	UtcTimingXSDateHttpServer   = "https://time.akamai.com"
	UtcTimingXSDateHttpServerMS = "https://time.akamai.com/?ms"
	UtcTimingISOHttpServer      = "https://time.akamai.com/?iso"
	UtcTimingISOHttpServerMS    = "https://time.akamai.com/?iso&ms"
	UtcTimingHeadAsset          = "/static/time.txt"
)

Should the xsdate and iso formats be swapped?

@tobbee
Copy link
Contributor

tobbee commented Jan 16, 2024

Reading the specs, I cannot see that there is any real difference between the xsdate and iso formats, so I will
make them both point to the ?iso URL variants.

@tobbee
Copy link
Contributor

tobbee commented Jan 19, 2024

The fix in #148 has been merged into v1.1.1 and published on https://livesim2.dashif.org

@tobbee tobbee closed this as completed Jan 19, 2024
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

2 participants