Skip to content

Remove uses of mbedtls_pk_verify_new #10449

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
Oct 24, 2025
Merged

Remove uses of mbedtls_pk_verify_new #10449

merged 1 commit into from
Oct 24, 2025
+3 −3

Conversation

@bjwtaylor
Copy link

@bjwtaylor bjwtaylor commented Oct 13, 2025 •
edited by mpg
Loading

Description

Remove uses of mbedtls_pk_verify_new

This PR is part of a multistage PR to be merged in the following order:

  1. Remove uses of mbedtls_pk_verify_new #10449
  2. Remove uses of mbedtls_pk_verify_new TF-PSA-Crypto#517

PR checklist

@bjwtaylor bjwtaylor mentioned this pull request Oct 13, 2025
Merged
4 tasks
@bjwtaylor bjwtaylor marked this pull request as ready for review October 14, 2025 06:43
@bjwtaylor bjwtaylor added 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 size-s Estimated task size: small (~2d) priority-high High priority - will be reviewed soon labels Oct 14, 2025
mpg
mpg approved these changes Oct 24, 2025
View reviewed changes
Copy link
Contributor

@mpg mpg left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@mpg mpg requested a review from valeriosetti October 24, 2025 10:55
@mpg mpg removed the needs-reviewer This PR needs someone to pick it up for review label Oct 24, 2025
@github-project-automation github-project-automation bot moved this from In Development to Has Approval in Roadmap pull requests (new board) Oct 24, 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 Oct 24, 2025
@valeriosetti valeriosetti added this pull request to the merge queue Oct 24, 2025
Merged via the queue into Mbed-TLS:development with commit 4624f50 Oct 24, 2025
7 of 8 checks passed
@github-project-automation github-project-automation bot moved this from Has Approval to Done in Roadmap pull requests (new board) Oct 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mpg mpg mpg 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 size-s Estimated task size: small (~2d)

Projects

Roadmap pull requests (new board)
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bjwtaylor @mpg @valeriosetti