Skip to content

[4.0.0] [WIP] ETR01SDK-584: Refactor lt_ecc_ecdsa_sign (remove hashing)#476

Draft
andreondra wants to merge 4 commits intodevelopfrom
ETR01SDK-584-Refactor-lt_ecc_ecdsa_sign-remove-hashing
Draft

[4.0.0] [WIP] ETR01SDK-584: Refactor lt_ecc_ecdsa_sign (remove hashing)#476
andreondra wants to merge 4 commits intodevelopfrom
ETR01SDK-584-Refactor-lt_ecc_ecdsa_sign-remove-hashing

Conversation

@andreondra
Copy link
Contributor

Description

As this PR changes the API behavior, it is draft until 4.0.0 release.

In this PR, I remove hashing from lt_ecc_ecdsa_sign. I also fix relevant test.


Type of Change

Select the type(s) that best describe your change:

  • 🐛 Bug fix
  • ✨ New feature
  • 🧹 Code cleanup or refactoring
  • 📝 Documentation update
  • 🔧 Build system or toolchain update
  • 🔒 Security improvement
  • Other (please describe):

Checklist

Before submitting, please confirm that you have completed the following:

  • I opened the Pull Request to the develop branch
  • I followed the project's code guidelines
  • I formatted the code using clang-format with the recommended configuration
  • I updated the changelog, or this change does not require it (e.g., internal or non-functional update)
  • The project builds without errors or warnings
  • I have verified the functionality against the hardware/model as applicable
  • I have ensured that public APIs remain backward compatible (if applicable)
  • This PR is ready for review by maintainers (no WIP commits left) and marked as Ready for Review

Optional Checks

You can enable optional CI jobs by checking following boxes. For example, coverage job is useful when modifying or implementing new tests.

  • Measure Test Coverage

@andreondra andreondra self-assigned this Feb 19, 2026
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.

1 participant