Skip to content

Remove: DES-related test, script, and selftest references #10269

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.

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

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

ariwo17
Copy link
Contributor

@ariwo17 ariwo17 commented Jul 2, 2025

Description

This PR removes several remaining references to DES from the Mbed TLS core repository as part of the larger DES removal tracked in #9164.

Specifically:

  • Removed DES self-test from programs/test/selftest.c
  • Removed DES detection logic from tests/compat.sh
  • Removed DES mentions from:
    • scripts/config.py (comment only)
    • tests/scripts/analyze_outcomes.py
    • tests/scripts/components-configuration-crypto.sh
    • tests/scripts/components-configuration-tls.sh

This PR does not update changelogs or documentation files. Those will be addressed separately after the full DES removal is complete.

Generated build artifacts (under build/) were not modified or committed.
All tests pass locally.

PR checklist

Please remove the segment/s on either side of the | symbol as appropriate, and add any relevant link/s to the end of the line.
If the provided content is part of the present PR remove the # symbol.

  • changelog not required because that will be provided in a follow-up commit after final DES cleanup.
  • development PR not required because this is a supporting PR contributing to issue Remove DES #9164.
  • TF-PSA-Crypto PR not required because this PR does not touch TF-PSA-Crypto.
  • framework PR not required
  • 3.6 PR not required because DES removal is only applicable to Mbed TLS 4.0 and later.
  • tests not required because no new tests were added; existing tests confirm a clean build.

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.

@ariwo17 ariwo17 self-assigned this Jul 3, 2025
@gilles-peskine-arm gilles-peskine-arm added needs-work size-s Estimated task size: small (~2d) priority-high High priority - will be reviewed soon labels Jul 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-work priority-high High priority - will be reviewed soon size-s Estimated task size: small (~2d)
Projects
Development

Successfully merging this pull request may close these issues.

2 participants