Count Amazon's time servers as time sync - #272
Merged
Merged
Conversation
Two Ring cameras showed up in Insights as "Started using a new app: Amazon" after they set their clocks from ntp-g7g.amazon.com, the time server Amazon's devices fall back to when their usual pool doesn't answer. The catalog gives every amazon.com name to the Amazon shopping app. ntp-g7g.amazon.com, and AWS's public time.aws.com, now belong to Time sync, which Insights leaves out of new app findings.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two Ring cameras showed up in Insights as Started using a new app: Amazon (Shopping) after they set their clocks from
ntp-g7g.amazon.com. That's the time server Amazon's devices fall back to when their usual*.amazon.pool.ntp.orgservers don't answer, which happened during a stretch of failed lookups this morning. The catalog gives everyamazon.comname to the Amazon shopping app.What changed
ntp-g7g.amazon.com, and AWS's publictime.aws.com, beside the Meta and Google time servers from Count brands' public time servers as time sync #267. Insights leaves Time sync out of new app findings.ntp-fireos.comstays out. Despite the name, it carries Amazon device metrics, not time.Lookupgivesntp-g7g.amazon.comto Time sync whilewww.amazon.comstays Amazon, and a device whose new lookups includentp-g7g.amazon.comgets no Amazon finding. Both fail against the old catalog.No console change, so no screenshots.
Checks
go tool mage verifyandgofmt -l .