Skip to content

Conversation

@tisj
Copy link

@tisj tisj commented Apr 14, 2023

Useful as an extra form of authentication in certificate enrollment/renewal using SCEP [1] or EST [2].

Specified in RFC 2985 [3] as

challengePassword ATTRIBUTE ::= {
        WITH SYNTAX DirectoryString {pkcs-9-ub-challengePassword}
        EQUALITY MATCHING RULE caseExactMatch
        SINGLE VALUE TRUE
        ID pkcs-9-at-challengePassword
}

Use mbedtls_x509write_csr_set_challenge_password function to set the challenge password in mbedtls_x509write_csr object, either as (text) printable (preferred), or UTF-8 string.

resolves #4886

  1. https://www.rfc-editor.org/rfc/rfc8894.html#name-challengepassword-shared-se
  2. https://www.rfc-editor.org/rfc/rfc7030#section-4.5
  3. https://www.rfc-editor.org/rfc/rfc2985#section-5.4

Description

Please write a few sentences describing the overall goals of the pull request's commits.

Gatekeeper checklist

  • changelog provided, or not required
  • backport done, or not required
  • tests provided, or not required

Notes for the submitter

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

@tisj tisj force-pushed the topic/support_csr_challenge_password branch from de465b7 to 4192491 Compare April 14, 2023 14:56
@yuhaoth yuhaoth added enhancement 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-m Estimated task size: medium (~1w) labels Apr 17, 2023
@yuhaoth
Copy link
Contributor

yuhaoth commented Apr 17, 2023

This PR got some code style fail . Please fix that.

Useful as an extra form of authentication in certificate
enrollment/renewal using SCEP [1] or EST [2].

Specified in RFC 2985 [3] as

challengePassword ATTRIBUTE ::= {
        WITH SYNTAX DirectoryString {pkcs-9-ub-challengePassword}
        EQUALITY MATCHING RULE caseExactMatch
        SINGLE VALUE TRUE
        ID pkcs-9-at-challengePassword
}

Use mbedtls_x509write_csr_set_challenge_password function to set the
challenge password in mbedtls_x509write_csr object, either as (text)
printable (preferred), or UTF-8 string.

resolves Mbed-TLS#4886

1. https://www.rfc-editor.org/rfc/rfc8894.html#name-challengepassword-shared-se
2. https://www.rfc-editor.org/rfc/rfc7030#section-4.5
3. https://www.rfc-editor.org/rfc/rfc2985#section-5.4

Signed-off-by: Tijs Van Buggenhout <tijs.van.buggenhout@axsguard.com>
@tisj tisj force-pushed the topic/support_csr_challenge_password branch from 4192491 to 8b9825e Compare April 17, 2023 09:24
@tisj
Copy link
Author

tisj commented Apr 17, 2023

@yuhaoth thx, done

Anything I should do about the ABI errors?

@daverodgman
Copy link
Contributor

Anything I should do about the ABI errors?

This is fine. The changes here are an ABI break but not an API break, which is OK for the 3.x branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement 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-m Estimated task size: medium (~1w)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ability to set challenge password in mbedtls_x509write_csr

3 participants