mbedtls icon indicating copy to clipboard operation
mbedtls copied to clipboard
verified publisher icon Mbed-TLS

Move the PAKE hash algorithm parameter into the alg id

Open amtkarm1 opened this issue 10 months ago • 1 comments

Description

Fixes Mbed-TLS/mbedtls#9321.

PR checklist

  • [x] changelog provided
  • [X] development PR provided HERE
  • [X] TF-PSA-Crypto PR provided Mbed-TLS/tf-psa-crypto#196
  • [X] framework PR not required
  • [X] 3.6 PR not required because: no backport required
  • [X] 2.28 PR not required because: no backport required
  • tests provided

amtkarm1 avatar Mar 10 '25 19:03 amtkarm1

@bensze01 Are you still reviewing this (and its crypto companion)? (Considering it's a new quarter and you've got new priorities.) It's perfectly OK if the answer is no, but then we need to know and find another reviewer.

mpg avatar Apr 24 '25 10:04 mpg

This has a conflict but it's only in the crypto pointer. I suggest we ignore it for now: we'll need to update the crypto pointer again when the crypto PR is merged, anyway - we'll resolve the conflict at that point.

mpg avatar Aug 07 '25 08:08 mpg

Sorry, actually my request for a rebase before we merge the TF-PSA-Crpyto PR was a bad idea because of the interdependencies between the repos. I've force-pushed the the pre-rebase version, with the latest TF-PSA-Crypto changes to make the CI happy.

bensze01 avatar Aug 12 '25 13:08 bensze01

I've restored Anton's rebase of this PR from yesterday.

bensze01 avatar Aug 13 '25 07:08 bensze01

We need to update the submodule pointer to the merge commit of https://github.com/Mbed-TLS/TF-PSA-Crypto/pull/196 - I'm doing this now.

mpg avatar Aug 13 '25 07:08 mpg