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.

Sign up for GitHub

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

Jump to bottom

Make Certificate messages beacon field optional #1807

Closed
2 tasks done
jpraynaud opened this issue Jul 8, 2024 · 0 comments · Fixed by #1839
Closed
2 tasks done

Make Certificate messages beacon field optional #1807

jpraynaud opened this issue Jul 8, 2024 · 0 comments · Fixed by #1839
Assignees
@Alenar @sfauvel
Labels
refactoring 🛠️ Code refactoring and enhancements

Comments

@jpraynaud
Copy link
Member

jpraynaud commented Jul 8, 2024 •
edited by Alenar
Loading

Why

While implementing the issue #1804, we have made the usage of the beacon field in the pending certificate optional. We want to do this operation on the Certificate messages for completeness.

What

Make the beacon field optional in the Certificate messages.

How

  • Make beacon field optional in CertificateMessage of Open API (make a test with a stable client)
  • Make beacon field optional in CertificateListItemMessage of Open API (make a test with a stable client)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Alenar Alenar

@sfauvel sfauvel

Labels
refactoring 🛠️ Code refactoring and enhancements
Projects
None yet
Milestone
No milestone
3 participants
@Alenar @sfauvel @jpraynaud