Skip to content
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

Replace MBEDTLS_MD_CAN_SHA384 with PSA_WANT_ALG_SHA_384 #9171

Merged
merged 2 commits into from
Jul 4, 2024
Merged

Replace MBEDTLS_MD_CAN_SHA384 with PSA_WANT_ALG_SHA_384 #9171

merged 2 commits into from
Jul 4, 2024

Conversation

eleuzi01
Copy link
Contributor

@eleuzi01 eleuzi01 commented May 23, 2024 •
edited
Loading

Description

Issue #9113

PR checklist

Please tick as appropriate and edit the reasons (e.g.: "backport: not needed because this is a new feature")

  • changelog not required, internal changes
  • 3.6 backport not required, needed for 4.0
  • 2.28 backport not required, needed for 4.0
  • tests not required

Notes for the submitter

Please refer to the contributing guidelines, especially the
checklist for PR contributors.

Help make review efficient:

  • Multiple simple commits
    • please structure your PR into a series of small commits, each of which does one thing
  • Avoid force-push
    • please do not force-push to update your PR - just add new commit(s)
  • See our Guidelines for Contributors for more details about the review process.

@eleuzi01 eleuzi01 added enhancement component-crypto Crypto primitives and low-level interfaces labels May 23, 2024
@eleuzi01 eleuzi01 self-assigned this May 23, 2024
@eleuzi01 eleuzi01 added the size-s Estimated task size: small (~2d) label May 31, 2024
@gabor-mezei-arm gabor-mezei-arm linked an issue Jun 5, 2024 that may be closed by this pull request
Replace MBEDTLS_MD_CAN_SHA384 with its PSA_WANT counterpartart #9113
Closed
@gabor-mezei-arm gabor-mezei-arm added the needs-preceding-pr Requires another PR to be merged first label Jun 5, 2024
@eleuzi01 eleuzi01 added needs-ci Needs to pass CI tests priority-high High priority - will be reviewed soon labels Jun 13, 2024
@eleuzi01 eleuzi01 force-pushed the replace-mbedtls-md-can-sha384 branch from ccadd54 to cc82cac Compare June 20, 2024 15:40
@eleuzi01 eleuzi01 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 and removed needs-preceding-pr Requires another PR to be merged first labels Jun 20, 2024
@gabor-mezei-arm gabor-mezei-arm self-requested a review June 20, 2024 16:07
@eleuzi01 eleuzi01 removed the needs-ci Needs to pass CI tests label Jun 24, 2024
gabor-mezei-arm
gabor-mezei-arm previously approved these changes Jun 28, 2024
View reviewed changes
Copy link
Contributor

@gabor-mezei-arm gabor-mezei-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

@eleuzi01 eleuzi01 force-pushed the replace-mbedtls-md-can-sha384 branch from 2f86570 to 94de429 Compare July 2, 2024 10:34
gabor-mezei-arm
gabor-mezei-arm previously approved these changes Jul 2, 2024
View reviewed changes
Copy link
Contributor

@gabor-mezei-arm gabor-mezei-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

@tom-daubney-arm tom-daubney-arm self-requested a review July 2, 2024 19:05
@tom-daubney-arm tom-daubney-arm removed the needs-reviewer This PR needs someone to pick it up for review label Jul 2, 2024
tom-daubney-arm
tom-daubney-arm previously approved these changes Jul 2, 2024
View reviewed changes
Copy link
Contributor

@tom-daubney-arm tom-daubney-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 - thanks!

@tom-daubney-arm tom-daubney-arm 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, approved Design and code approved - may be waiting for CI or backports labels Jul 2, 2024
@tom-daubney-arm
Copy link
Contributor

I have just removed the approved label and added DO-NOT-MERGE because this now needs a rebase and some conflict resolution.

@eleuzi01
Replace MBEDTLS_MD_CAN_SHA384 with PSA_WANT_ALG_SHA_384
b476d4b 
Signed-off-by: Elena Uziunaite <elena.uziunaite@arm.com>
@eleuzi01
Replace MBEDTLS_MD_CAN_SHA384 in md.h and ssl.h
cf5e3dd 
Signed-off-by: Elena Uziunaite <elena.uziunaite@arm.com>
@eleuzi01 eleuzi01 force-pushed the replace-mbedtls-md-can-sha384 branch from 94de429 to cf5e3dd Compare July 3, 2024 09:21
@tom-daubney-arm tom-daubney-arm added needs-review Every commit must be reviewed by at least two team members, needs-ci Needs to pass CI tests and removed DO-NOT-MERGE labels Jul 3, 2024
@tom-daubney-arm tom-daubney-arm self-requested a review July 3, 2024 09:29
tom-daubney-arm
tom-daubney-arm approved these changes Jul 3, 2024
View reviewed changes
Copy link
Contributor

@tom-daubney-arm tom-daubney-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 - subject to CI - thanks!

@gabor-mezei-arm
Copy link
Contributor

The legacy macro removal PRs most probably cause a merge conflict to each other so wait for the others to be merged.

gabor-mezei-arm
gabor-mezei-arm approved these changes Jul 4, 2024
View reviewed changes
Copy link
Contributor

@gabor-mezei-arm gabor-mezei-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

@ronald-cron-arm ronald-cron-arm added this pull request to the merge queue Jul 4, 2024
@gabor-mezei-arm gabor-mezei-arm 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, needs-ci Needs to pass CI tests labels Jul 4, 2024
Merged via the queue into Mbed-TLS:development with commit 2cf41a2 Jul 4, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tom-daubney-arm tom-daubney-arm tom-daubney-arm approved these changes

@gabor-mezei-arm gabor-mezei-arm gabor-mezei-arm approved these changes

Assignees

@eleuzi01 eleuzi01

Labels
approved Design and code approved - may be waiting for CI or backports component-crypto Crypto primitives and low-level interfaces enhancement priority-high High priority - will be reviewed soon size-s Estimated task size: small (~2d)
Projects
Roadmap Board for Mbed TLS
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Replace MBEDTLS_MD_CAN_SHA384 with its PSA_WANT counterpartart
4 participants
@eleuzi01 @tom-daubney-arm @gabor-mezei-arm @ronald-cron-arm