Skip to content

Remove support for static ECDH cipher suites #182

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
Merged
merged 1 commit into from
Jul 15, 2025
Merged

Remove support for static ECDH cipher suites #182

merged 1 commit into from
Jul 15, 2025

Conversation

bjwtaylor
Copy link

@bjwtaylor bjwtaylor commented Jul 10, 2025 •
edited
Loading

Description

Remove support for static ECDH cipher suites, contributes Mbed-TLS/mbedtls#9201

This PR is part of a chain which needs to be merged in the following order:

  1. Remove support for static ECDH cipher suites #182
  2. Remove support for static ECDH cipher suites mbedtls#10294

PR checklist

  • TF-PSA-Crypto PR not required because: No changes
  • development PR provided #HERE
  • 3.6 PR not required because: No backports

@bjwtaylor bjwtaylor mentioned this pull request Jul 10, 2025
Merged
5 tasks
@bjwtaylor bjwtaylor added needs-ci Needs to pass CI tests needs-review Every commit must be reviewed by at least two team members, needs-reviewer This PR needs someone to pick it up for review and removed needs-ci Needs to pass CI tests labels Jul 10, 2025
@bjwtaylor bjwtaylor marked this pull request as ready for review July 11, 2025 07:25
gilles-peskine-arm
gilles-peskine-arm approved these changes Jul 11, 2025
View reviewed changes
Copy link
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

(Normally things in the framework should stay because of 3.6, but we don't have psasim in 3.6.)

@gilles-peskine-arm gilles-peskine-arm added the priority-high High priority - will be reviewed soon label Jul 11, 2025
@valeriosetti valeriosetti self-requested a review July 15, 2025 10:49
@valeriosetti valeriosetti removed the needs-reviewer This PR needs someone to pick it up for review label Jul 15, 2025
@github-project-automation github-project-automation bot moved this from In Development to Has Approval in Roadmap pull requests (new board) Jul 15, 2025
@valeriosetti valeriosetti added approved Design and code approved - may be waiting for CI or backports and removed needs-review Every commit must be reviewed by at least two team members, labels Jul 15, 2025
@gilles-peskine-arm gilles-peskine-arm merged commit d5d1629 into Mbed-TLS:main Jul 15, 2025
3 of 4 checks passed
@github-project-automation github-project-automation bot moved this from Has Approval to Done in Roadmap pull requests (new board) Jul 15, 2025
@bjwtaylor bjwtaylor mentioned this pull request Jul 16, 2025
Merged
4 tasks
@bjwtaylor bjwtaylor mentioned this pull request Aug 21, 2025
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gilles-peskine-arm gilles-peskine-arm gilles-peskine-arm approved these changes

@valeriosetti valeriosetti valeriosetti approved these changes

Assignees
No one assigned
Labels
approved Design and code approved - may be waiting for CI or backports priority-high High priority - will be reviewed soon
Projects
Roadmap pull requests (new board)
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bjwtaylor @gilles-peskine-arm @valeriosetti