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

use timesource(synced) to generate/validate server cert time #4228

Merged
merged 3 commits into from
Nov 7, 2023

Conversation

qfrank
Copy link
Contributor

@qfrank qfrank commented Oct 30, 2023

major changes:

  • use timesource to fill the time of the cert
  • use timesource to validate cert time

with this PR, we can account for potential time inaccuracies in devices, ensuring that certificate validation is not adversely affected by devices with incorrect system times.

relate comment

status: ready.

@status-im-auto
Copy link
Member

status-im-auto commented Oct 30, 2023

Jenkins Builds

Click to see older builds (19)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ ec97d9e #1 2023-10-30 14:13:41 ~3 min linux 📦zip
✔️ ec97d9e #1 2023-10-30 14:15:31 ~5 min android 📦aar
✔️ ec97d9e #1 2023-10-30 14:44:30 ~34 min tests 📄log
✔️ d6bafce #2 2023-10-30 14:17:19 ~1 min android 📦aar
✔️ d6bafce #2 2023-10-30 14:17:36 ~3 min linux 📦zip
✔️ d6bafce #2 2023-10-30 14:26:44 ~5 min ios 📦zip
✖️ d6bafce #2 2023-10-30 15:11:34 ~26 min tests 📄log
✔️ 468598e #3 2023-10-31 03:41:07 ~1 min linux 📦zip
✔️ 468598e #3 2023-10-31 03:42:42 ~3 min ios 📦zip
✔️ 468598e #3 2023-10-31 03:45:00 ~5 min android 📦aar
✔️ 468598e #3 2023-10-31 04:13:01 ~33 min tests 📄log
✔️ 9270352 #4 2023-11-01 13:08:26 ~2 min android 📦aar
✖️ 9270352 #4 2023-11-01 13:08:51 ~2 min tests 📄log
✔️ 9270352 #4 2023-11-01 13:09:22 ~3 min ios 📦zip
✔️ 9270352 #4 2023-11-01 13:09:24 ~3 min linux 📦zip
✔️ 70cd023 #5 2023-11-02 05:06:52 ~2 min android 📦aar
✔️ 70cd023 #5 2023-11-02 05:07:35 ~3 min linux 📦zip
✔️ 70cd023 #5 2023-11-02 05:07:53 ~3 min ios 📦zip
✔️ 70cd023 #5 2023-11-02 05:37:22 ~32 min tests 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ d196eea #6 2023-11-02 12:28:03 ~1 min linux 📦zip
✔️ d196eea #6 2023-11-02 12:31:47 ~5 min android 📦aar
✖️ d196eea #6 2023-11-02 12:43:36 ~17 min tests 📄log
✔️ 35c3d6d #7 2023-11-06 14:19:50 ~1 min linux 📦zip
✔️ 35c3d6d #7 2023-11-06 14:21:54 ~3 min ios 📦zip
✔️ 35c3d6d #7 2023-11-06 14:23:13 ~4 min android 📦aar
✖️ 35c3d6d #7 2023-11-06 14:32:29 ~14 min tests 📄log
✖️ 35c3d6d #8 2023-11-06 15:11:41 ~33 min tests 📄log
✖️ 35c3d6d #9 2023-11-06 15:21:09 ~4 min tests 📄log
✖️ 35c3d6d #10 2023-11-06 15:55:54 ~32 min tests 📄log
✖️ 35c3d6d #11 2023-11-06 16:32:07 ~32 min tests 📄log
✖️ 35c3d6d #12 2023-11-07 00:43:47 ~3 min tests 📄log
✔️ 35c3d6d #13 2023-11-07 01:18:23 ~32 min tests 📄log

@qfrank qfrank force-pushed the chore/use-timesource-to-generate-cert branch from ec97d9e to d6bafce Compare October 30, 2023 14:13
Copy link
Collaborator

@igor-sirotin igor-sirotin left a comment

Choose a reason for hiding this comment

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

nice 👍

@qfrank qfrank force-pushed the chore/use-timesource-to-generate-cert branch 3 times, most recently from 9270352 to 70cd023 Compare November 2, 2023 05:04
@qfrank qfrank force-pushed the chore/use-timesource-to-generate-cert branch from d196eea to 35c3d6d Compare November 6, 2023 14:18
@qfrank qfrank merged commit 7ad5800 into develop Nov 7, 2023
7 checks passed
@qfrank qfrank deleted the chore/use-timesource-to-generate-cert branch November 7, 2023 01:51
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.

4 participants