Merge pull request #7387 from bhandras/cert-fixup

cert: removed unused args from `cert.GenCertPair` [no-itest]

cert/selfsigned.go

@@ -200,9 +200,9 @@ func IsOutdated(cert *x509.Certificate, tlsExtraIPs,
 //
 // This function is adapted from https://github.com/btcsuite/btcd and
 // https://github.com/btcsuite/btcd/btcutil
-func GenCertPair(org, certFile, keyFile string, tlsExtraIPs,
-	tlsExtraDomains []string, tlsDisableAutofill bool,
-	certValidity time.Duration) ([]byte, []byte, error) {
+func GenCertPair(org string, tlsExtraIPs, tlsExtraDomains []string,
+	tlsDisableAutofill bool, certValidity time.Duration) (
+	[]byte, []byte, error) {
 
 	now := time.Now()
 	validUntil := now.Add(certValidity)

cert/selfsigned_test.go

@@ -29,8 +29,8 @@ func TestIsOutdatedCert(t *testing.T) {
 
 	// Generate TLS files with two extra IPs and domains.
 	certBytes, keyBytes, err := cert.GenCertPair(
-		"lnd autogenerated cert", certPath, keyPath, extraIPs[:2],
-		extraDomains[:2], false, testTLSCertDuration,
+		"lnd autogenerated cert", extraIPs[:2], extraDomains[:2],
+		false, testTLSCertDuration,
 	)
 	if err != nil {
 		t.Fatal(err)
@@ -89,8 +89,8 @@ func TestIsOutdatedPermutation(t *testing.T) {
 
 	// Generate TLS files from the IPs and domains.
 	certBytes, keyBytes, err := cert.GenCertPair(
-		"lnd autogenerated cert", certPath, keyPath, extraIPs[:],
-		extraDomains[:], false, testTLSCertDuration,
+		"lnd autogenerated cert", extraIPs[:], extraDomains[:],
+		false, testTLSCertDuration,
 	)
 	if err != nil {
 		t.Fatal(err)
@@ -160,8 +160,8 @@ func TestTLSDisableAutofill(t *testing.T) {
 
 	// Generate TLS files with two extra IPs and domains and no interface IPs.
 	certBytes, keyBytes, err := cert.GenCertPair(
-		"lnd autogenerated cert", certPath, keyPath, extraIPs[:2],
-		extraDomains[:2], true, testTLSCertDuration,
+		"lnd autogenerated cert", extraIPs[:2], extraDomains[:2],
+		true, testTLSCertDuration,
 	)
 	require.NoError(
 		t, err,
@@ -222,9 +222,8 @@ func TestTLSConfig(t *testing.T) {
 
 	// Generate TLS files with an extra IP and domain.
 	certBytes, keyBytes, err := cert.GenCertPair(
-		"lnd autogenerated cert", certPath, keyPath,
-		[]string{extraIPs[0]}, []string{extraDomains[0]}, false,
-		testTLSCertDuration,
+		"lnd autogenerated cert", []string{extraIPs[0]},
+		[]string{extraDomains[0]}, false, testTLSCertDuration,
 	)
 	require.NoError(t, err)
 

docs/release-notes/release-notes-0.16.0.md

@@ -258,6 +258,9 @@ in the lnwire package](https://github.com/lightningnetwork/lnd/pull/7303)
 * [Add a flag](https://github.com/lightningnetwork/lnd/pull/6527) to allow for
   the option to encrypt the tls key.
 
+* [Remove unused args](https://github.com/lightningnetwork/lnd/pull/7387)
+  from `cert.GenCertPair`.
+
 ## `lncli`
 
 * [Add an `insecure` flag to skip tls auth as well as a `metadata` string slice