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
APPS: generalize load_csr()
and remove obsolete OBJ_create()
#18900
Conversation
…so for apps, too Also add related references to FR openssl#15725.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Asking @openssl/committers for 2nd review. |
Also here, ping https://github.com/orgs/openssl/teams/committers for 2nd approval. |
Ping @openssl/committers for 2nd review. |
Hmm, I am not sure about this as this changes the behavior to not enforce the PEM/DER format but just try first the format from the option. For X.509s we do not enforce the format by default but once the -inform is used it is enforced. |
This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago |
…e it also for apps, too
Oh, @t8m I just noticed that I overlooked your comment so far. |
The failing
|
@beldmit still OK? |
Yes, still OK. |
This pull request is ready to merge |
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com> (Merged from #18900)
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com> (Merged from openssl#18900)
…so for apps, too Also add related references to FR openssl#15725. Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com> (Merged from openssl#18900)
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com> (Merged from openssl#18900)
…so for apps, too Also add related references to FR openssl#15725. Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com> (Merged from openssl#18900)
This has been carved out of #16006 and slightly extended.
load_csr_autofmt()
fromapps/cmp.c
toapps.c
and use it also for other appsapps/x509.c:
Remove legacy call toOBJ_create()