Added options to x509 tool

[ecdeye]

Description:

Contains initial implementation for OpenSSL x509 tool, options -in -out, -req, -signkey, -modulus, -days, -dates,
-checkend, -noout (x509.cc), as well as testing for option functionality and argument requirements (x509_test.cc)

Files expected to change

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 81.42292% with 47 lines in your changes missing coverage. Please review.

Project coverage is 78.27%. Comparing base (1fbf584) to head (0d38598).
Report is 2 commits behind head on main.

Files	Patch %	Lines
tool-openssl/x509.cc	70.44%	47 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1696      +/-   ##
==========================================
+ Coverage   78.22%   78.27%   +0.05%     
==========================================
  Files         571      571              
  Lines       95534    95739     +205     
  Branches    13701    13736      +35     
==========================================
+ Hits        74729    74941     +212     
  Misses      20195    20195              
+ Partials      610      603       -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[samuel40791765]

Great work and test coverage! Just a few minor comments here and there

[samuel40791765]

Great test! It does seem like we can parameterize the args options here with TEST_P and C++ classes. It's just a potential improvement, but not really a priority. This might serve as a good example:

aws-lc/crypto/cipher_extra/cipher_test.cc

Lines 591 to 616 in 1fbf584

	struct AeadCipherParams {
	const char name[40];
	const EVP_CIPHER *(*func)(void);
	const char *test_vectors;
	};
	static const struct AeadCipherParams AeadCiphers[] = {
	{"ChaCha20Poly1305", EVP_chacha20_poly1305, "chacha20_poly1305_tests.txt"},
	{"AES_128_CCM_BLUETOOTH", EVP_aes_128_ccm, "aes_128_ccm_bluetooth_tests.txt"},
	{"AES_128_CCM_BLUETOOTH_8", EVP_aes_128_ccm,
	"aes_128_ccm_bluetooth_8_tests.txt"},
	{"AES_128_CCM_Matter", EVP_aes_128_ccm, "aes_128_ccm_matter_tests.txt"},
	};
	class AeadCipherTest : public testing::TestWithParam<AeadCipherParams> {
	public:
	const EVP_CIPHER *getTestCipher() {
	return GetParam().func();
	}
	};
	INSTANTIATE_TEST_SUITE_P(All, AeadCipherTest, testing::ValuesIn(AeadCiphers),
	[](const testing::TestParamInfo<AeadCipherParams> &params)
	-> std::string { return params.param.name; });
	TEST_P(AeadCipherTest, TestVector) {

[ecdeye]

Currently trying to implement but errors preventing build. Will come back to this later