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.

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

Remove DES #9164

Open
gilles-peskine-arm opened this issue May 22, 2024 · 4 comments
Open

Remove DES #9164

gilles-peskine-arm opened this issue May 22, 2024 · 4 comments
Labels
api-break This issue/PR breaks the API and must wait for a new major version component-crypto Crypto primitives and low-level interfaces size-s Estimated task size: small (~2d)

Comments

@gilles-peskine-arm
Copy link
Contributor

gilles-peskine-arm commented May 22, 2024

Remove DES from the next major version of Mbed TLS.

Rationale: DES, even 3DES, is no longer in common use. NIST no longer allows it except to process legacy data since the withdrawal of SP 800-67r2 in January 2024. DES remains in use mainly in some banking applications, where Mbed TLS is not commonly used.

This is broader than #4396 and easier to do.

Mailing list thread: https://lists.trustedfirmware.org/archives/list/[email protected]/thread/26ZWV67A4ECK2D5LSPJEDTT22MLOJOMQ/

@gilles-peskine-arm gilles-peskine-arm added needs-design-approval component-crypto Crypto primitives and low-level interfaces api-break This issue/PR breaks the API and must wait for a new major version size-s Estimated task size: small (~2d) labels May 22, 2024
@tom-cosgrove-arm
Copy link
Contributor

Let's rename this to just "Remove DES from Mbed TLS", make it a MUST for 4.0, and close #4396 as superseded by this one.

Having DES completely gone is easier for consumers of the library - anyone who requires any size of DES can just use the LTS.

@gilles-peskine-arm
Copy link
Contributor Author

Architectural decision: we are going to remove DES in TF-PSA-Crypto 1.0.

@davidhorstmann-arm
Copy link
Contributor

DES remains in use mainly in some banking applications, where Mbed TLS is not commonly used.

It does not alter this decision since this is not our focus, but noting for interest: we have received a contribution for z/OS, commonly used on banking mainframes, as I understand.

@gilles-peskine-arm
Copy link
Contributor Author

Different banking, AFAIK. z/OS is a mainframe platform that decides how much money you have on your account. DES is used in banking cards and devices that communicate with banking cards (there's an ongoing transition to AES, but it's not quite finished yet).

@gilles-peskine-arm gilles-peskine-arm changed the title Consider removing DES Remove DES Aug 14, 2024
@yanesca yanesca moved this to Removals and deprecations in Mbed TLS Backlog Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-break This issue/PR breaks the API and must wait for a new major version component-crypto Crypto primitives and low-level interfaces size-s Estimated task size: small (~2d)
Projects
Status: Implementation needed
Status: No status
Development

No branches or pull requests

3 participants