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

EventProducer - rely on true time instead of the system clock #14

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

paulinaaniola
Copy link
Contributor

For each event, we add a requested-sent-timestamp header. Instead of relying on the system clock, we should rely on the true time. I used the same library as in the player module.

@paulinaaniola paulinaaniola force-pushed the paulina/event-producer-use-true-time branch from c27baf9 to c1dc9cb Compare June 11, 2024 11:01
Copy link
Contributor

@kgrevehagen kgrevehagen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@paulinaaniola paulinaaniola added this pull request to the merge queue Jun 11, 2024
Merged via the queue into main with commit 289f5a2 Jun 11, 2024
7 checks passed
@paulinaaniola paulinaaniola deleted the paulina/event-producer-use-true-time branch June 11, 2024 11:34
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

Successfully merging this pull request may close these issues.

2 participants