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

Enhancement/2090 Improve Sgx Logs #2096

Open
wants to merge 3 commits into
base: v4.0.0
Choose a base branch
from
Open

Enhancement/2090 Improve Sgx Logs #2096

wants to merge 3 commits into from

Conversation

PropzSaladaz
Copy link

Description

  • Added error logs for SGX non-existent BLS & ECDSA key names
  • Improved log when blocks cannot be deserialized due to incorrect BLS key shares

Tests

  • Tested in local servers

@PropzSaladaz PropzSaladaz self-assigned this Feb 6, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 6, 2025

SUGGESTIONS BEFORE MERGE:

  1. Build custom container here: https://github.com/skalenetwork/skaled/actions?query=workflow%3A%22Custom+Docker+and+Binary%22
  2. Test it here: https://github.com/skalenetwork/skale-ci-integration_tests/actions?query=workflow%3A%22Skaled+Nightly+Tests%22
  3. Edit VERSION file appropriately to your changes!
    See https://skalelabs.atlassian.net/wiki/spaces/SKALE/pages/2444951553/Branching+instructions

@PropzSaladaz PropzSaladaz changed the base branch from develop to v4.0.0 February 6, 2025 11:45
@PropzSaladaz PropzSaladaz linked an issue Feb 6, 2025 that may be closed by this pull request
Add more informative SGX related logs skalenetwork/skale-consensus#841
Closed
@DmytroNazarenko DmytroNazarenko added this to the SKALE 4.0 milestone Feb 7, 2025
@PropzSaladaz PropzSaladaz changed the title Enhancement/2090 Improve Sgx Logs Enhancement/2090 Improve Sgx Logs [DO NOT MERGE] Feb 10, 2025
@PropzSaladaz PropzSaladaz changed the title Enhancement/2090 Improve Sgx Logs [DO NOT MERGE] Enhancement/2090 Improve Sgx Logs Feb 10, 2025
@olehnikolaiev olehnikolaiev self-requested a review February 10, 2025 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@olehnikolaiev olehnikolaiev olehnikolaiev approved these changes

@DmytroNazarenko DmytroNazarenko Awaiting requested review from DmytroNazarenko DmytroNazarenko is a code owner

@kladkogex kladkogex Awaiting requested review from kladkogex kladkogex is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

@PropzSaladaz PropzSaladaz

Labels
None yet
Projects
None yet
Milestone
SKALE 4.0
Development

Successfully merging this pull request may close these issues.

Add more informative SGX related logs
3 participants
@PropzSaladaz @olehnikolaiev @DmytroNazarenko