-
Notifications
You must be signed in to change notification settings - Fork 39
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
Make deprecated beacon
field optional in Certificate
& CertificateList
messages
#1839
Conversation
…essage to entity It used the deprecated beacon instead of using the `CertificateMetadataMessagePart` network field that's the updated, long-term, field to use. Co-authored-by: Sébastien Fauvel <sfauvel@users.noreply.github.com>
So we can remove it gracefully in the future when enough nodes have updated. Co-authored-by: Sébastien Fauvel <sfauvel@users.noreply.github.com>
So we can remove it gracefully in the future when enough nodes have updated. Co-authored-by: Sébastien Fauvel <sfauvel@users.noreply.github.com>
25ca517
to
61e1ba0
Compare
Co-authored-by: Sébastien Fauvel <sfauvel@users.noreply.github.com>
cd52839
to
10c1600
Compare
* Mithril-aggregator from `0.5.45` to `0.5.46` * Mithril-client-cli from `0.9.6` to `0.9.7` * Mithril-common from `0.4.32` to `0.4.33` * OpenApi from `0.1.26` to `0.1.27`
10c1600
to
3be9550
Compare
Test Results 4 files ±0 52 suites ±0 9m 2s ⏱️ -1s Results for commit 3be9550. ± Comparison against base commit 1375bce. This pull request removes 2 and adds 6 tests. Note that renamed tests count towards both.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Content
This PR make the, deprecated,
beacon
field an Option in theCertificate
&CertificateList
messages.This will allow us to remove them completely in the future when enough nodes have been updated, making the switch to "variable" beacon, aka SignedEntityType, complete.
Pre-submit checklist
Issue(s)
Closes #1807