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

Ability to set SntpClient pool interval #1784

Closed
nikitapn opened this issue Oct 7, 2024 · 1 comment
Closed

Ability to set SntpClient pool interval #1784

nikitapn opened this issue Oct 7, 2024 · 1 comment
Assignees

Comments

@nikitapn
Copy link

nikitapn commented Oct 7, 2024

Use case description

We are playing dash content with the absence of UTCTiming element in the manifests.
Because of that, SntpClient is used as wall clock time source. There is no option to insert UTCTiming on server side.
Our app never unloads itself from memory since it runs as launcher.
After a while (approximately a week) there is a significant drift in SystemClock.elapsedRealtime(), which makes player to do inaccurate requests for media segments which leads to player stops playing live content, since it requests segments that do not exist yet on the server.

Proposed solution

It would be great to have a new method for SntpClient that allows for setting a custom synchronization interval or leaving it unset.
Something like:

SntpClient.setNtpPoolInterval(12L * 3600L * 1000L)

Proposed example: 44751be

--
There is already a similar issue: #697
But the sync interval is hardcoded to 10 minutes, We don't think It needs to be so frequent, and It would be much better to have an option to set a desired synchronization interval in runtime.

@marcbaechinger
Copy link
Contributor

Thanks for your suggestion.

Yeah, that sound like a reasonable request. I'm going to take the pull #1794 that allows reinitialization.

I'd appreciate if you'd join the discussion there! I close this issue instead! Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants