refactor: Use ECDSA instead of RSA for TLS tests #133

Merged
nemith merged 1 commit from brb/push-uyynmqtkttun into main 2026-01-08 17:00:49 +00:00
nemith commented 2026-01-08 16:58:42 +00:00 (Migrated from github.com)

ECDSA is quite a bit faster and we aren't really testing the security (however it should be just as good if not better)

ECDSA is quite a bit faster and we aren't really testing the security (however it should be just as good if not better)
copilot-pull-request-reviewer[bot] (Migrated from github.com) reviewed 2026-01-08 17:00:03 +00:00
copilot-pull-request-reviewer[bot] (Migrated from github.com) left a comment

Pull request overview

This PR refactors the TLS test certificate generation to use ECDSA instead of RSA for improved test performance. ECDSA P-256 provides equivalent security to RSA 2048-bit but with faster key generation, which is beneficial for test execution speed.

  • Changed cryptographic algorithm from RSA 2048-bit to ECDSA P-256 for test certificate generation
  • Updated imports to reflect the new cryptographic dependencies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

## Pull request overview This PR refactors the TLS test certificate generation to use ECDSA instead of RSA for improved test performance. ECDSA P-256 provides equivalent security to RSA 2048-bit but with faster key generation, which is beneficial for test execution speed. - Changed cryptographic algorithm from RSA 2048-bit to ECDSA P-256 for test certificate generation - Updated imports to reflect the new cryptographic dependencies --- 💡 <a href="/nemith/netconf/new/main/.github/instructions?filename=*.instructions.md" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Add Copilot custom instructions</a> for smarter, more guided reviews. <a href="https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Learn how to get started</a>.
Sign in to join this conversation.
No description provided.