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

openssl_csr_info - add key size if possible #227

Closed
markatdxb opened this issue May 9, 2021 · 4 comments · Fixed by #233
Closed

openssl_csr_info - add key size if possible #227

markatdxb opened this issue May 9, 2021 · 4 comments · Fixed by #233

Comments

@markatdxb
Copy link

SUMMARY

Would be possible to add the key size and key algorithm info into module results?

ISSUE TYPE
  • Feature Idea
    we use this this module to validate user submitted CSRs in automation process and this would help to validate correct key size and selected algorithm
COMPONENT NAME

openssl_csr_info

ADDITIONAL INFORMATION

Here is a sample extract

openssl req -in /tmp/test.csr -noout -text

Certificate Request:
    Data:
        Version: 1 (0x0)
        Subject: C = US, L = Chicago, O = ACME, OU = IT, CN = testcert
        Subject Public Key Info:
            Public Key Algorithm: rsaEncryption
                RSA Public-Key: (2048 bit)
                Modulus:
                    00:f0:c9:ea:ac:ef:09:64:d

@felixfontein
Copy link
Contributor

(Needs #204 to be merged first.)

@felixfontein
Copy link
Contributor

I think it would be best to implement a openssl_public_key_info module first (in a generic way), and then use it to provide information on public keys for both openssl_csr_info and x509_certificate_info.

@felixfontein
Copy link
Contributor

Once #231 is reviewed and merged, I'll add public key information to openssl_csr_info and x509_certificate_info.

@felixfontein
Copy link
Contributor

Implemented in #233.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants