[FAB-3503] Wrong MSP keystore directory location

See [FAB-3503].
The keystore location is in the wrong location.
It is being created in the current working directory, but should
be in the client's HOME/msp directory.

Change-Id: I15d7714a2c7be128cded74d288d062b8d7b96b8a
Signed-off-by: Keith Smith <[email protected]>

cmd/fabric-ca-client/config.go

@@ -150,7 +150,7 @@ bccsp:
         security: 256
         filekeystore:
             # The directory used for the software file-based keystore
-            keystore: keystore
+            keystore: msp/keystore
 `
 )
 

cmd/fabric-ca-server/main_test.go

@@ -263,9 +263,9 @@ func TestClean(t *testing.T) {
 	os.Remove(unsupportedFileType)
 	os.Remove("ca-key.pem")
 	os.Remove("ca-cert.pem")
-	os.Remove("ca-cert.pem")
 	os.Remove("fabric-ca-server.db")
-	os.RemoveAll("msp")
+	os.RemoveAll("keystore")
+	os.RemoveAll("../../testdata/keystore")
 	os.Remove("../../testdata/fabric-ca-server.db")
 	os.Remove("../../testdata/ca-cert.pem")
 }

lib/ca.go

@@ -112,8 +112,7 @@ func (ca *CA) init(renew bool) (err error) {
 		return err
 	}
 	// Initialize the crypto layer (BCCSP) for this CA
-	defaultKeyStoreDir := path.Join(ca.HomeDir, "msp", "keystore")
-	ca.csp, err = csp.InitBCCSP(&ca.Config.CSP, defaultKeyStoreDir)
+	ca.csp, err = csp.InitBCCSP(&ca.Config.CSP, ca.HomeDir)
 	if err != nil {
 		return err
 	}

lib/client.go

@@ -89,8 +89,7 @@ func (c *Client) Init() error {
 			return fmt.Errorf("Failed to create cacerts directory: %s", err)
 		}
 		// Initialize BCCSP (the crypto layer)
-		keyStoreDir := path.Join(mspDir, "keystore")
-		c.csp, err = csp.InitBCCSP(&cfg.CSP, keyStoreDir)
+		c.csp, err = csp.InitBCCSP(&cfg.CSP, c.HomeDir)
 		if err != nil {
 			return err
 		}

lib/csp/util.go

@@ -24,6 +24,8 @@ import (
 	"errors"
 	"fmt"
 	"io/ioutil"
+	"path"
+	"path/filepath"
 	"strings"
 	_ "time" // for ocspSignerFromConfig
 
@@ -47,9 +49,10 @@ func GetDefaultBCCSP() bccsp.BCCSP {
 }
 
 // InitBCCSP initializes BCCSP
-func InitBCCSP(optsPtr **factory.FactoryOpts, keyStoreDir string) (bccsp.BCCSP, error) {
+func InitBCCSP(optsPtr **factory.FactoryOpts, homeDir string) (bccsp.BCCSP, error) {
 	// Initialize the config, setting defaults as needed
 	var opts *factory.FactoryOpts
+	var err error
 	if optsPtr != nil {
 		opts = *optsPtr
 	}
@@ -64,11 +67,16 @@ func InitBCCSP(optsPtr **factory.FactoryOpts, keyStoreDir string) (bccsp.BCCSP,
 		if opts.SwOpts.FileKeystore == nil ||
 			opts.SwOpts.FileKeystore.KeyStorePath == "" {
 			opts.SwOpts.Ephemeral = false
-			opts.SwOpts.FileKeystore = &factory.FileKeystoreOpts{KeyStorePath: keyStoreDir}
+			opts.SwOpts.FileKeystore = &factory.FileKeystoreOpts{KeyStorePath: path.Join("msp", "keystore")}
 		}
+		opts.SwOpts.FileKeystore.KeyStorePath, err = makeFileAbs(opts.SwOpts.FileKeystore.KeyStorePath, homeDir)
+		if err != nil {
+			return nil, fmt.Errorf("Failed to initialize BCCSP: %s", err)
+		}
+		log.Debugf("Software key file store directory: %s", opts.SwOpts.FileKeystore.KeyStorePath)
 	}
 	// Init the BCCSP factories
-	err := factory.InitFactories(opts)
+	err = factory.InitFactories(opts)
 	if err != nil {
 		return nil, fmt.Errorf("Failed to initialize BCCSP Factories: %s", err)
 	}
@@ -236,3 +244,18 @@ func ImportBCCSPKeyFromPEM(keyFile string, myCSP bccsp.BCCSP, temporary bool) (b
 		return nil, fmt.Errorf("Failed to import key from %s: invalid secret key type", keyFile)
 	}
 }
+
+// makeFileAbs makes 'file' absolute relative to 'dir' if not already absolute
+func makeFileAbs(file, dir string) (string, error) {
+	if file == "" {
+		return "", nil
+	}
+	if filepath.IsAbs(file) {
+		return file, nil
+	}
+	path, err := filepath.Abs(filepath.Join(dir, file))
+	if err != nil {
+		return "", fmt.Errorf("Failed making '%s' absolute based on '%s'", file, dir)
+	}
+	return path, nil
+}

lib/server_test.go

@@ -700,6 +700,7 @@ func TestEnd(t *testing.T) {
 	os.RemoveAll(rootDir)
 	os.RemoveAll(intermediateDir)
 	os.RemoveAll("multica")
+	os.RemoveAll(serversDir)
 	cleanMultiCADir()
 }