imp(keyring): improve UX for keyring.List (#13369)

* imp(keyring): improve UX for keyring.List

* use offline info in case of error

* Delete .dccache

* c++

* conflicts

.gitignore

@@ -6,6 +6,7 @@
 *.swm
 *.swn
 *.pyc
+.dccache
 
 # private files
 private[.-]*

crypto/keyring/keyring.go

@@ -470,29 +470,48 @@ func wrapKeyNotFound(err error, msg string) error {
 }
 
 func (ks keystore) List() ([]Info, error) {
-	var res []Info
+	res := []Info{}
 
 	keys, err := ks.db.Keys()
 	if err != nil {
 		return nil, err
 	}
 
-	sort.Strings(keys)
+	if len(keys) == 0 {
+		return res, nil
+	}
 
+	sort.Strings(keys)
 	for _, key := range keys {
 		if strings.HasSuffix(key, infoSuffix) {
 			rawInfo, err := ks.db.Get(key)
 			if err != nil {
-				return nil, err
+				fmt.Printf("err for key %s: %q\n", key, err)
+
+				// add the name of the key in case the user wants to retrieve it
+				// afterwards
+				info := newOfflineInfo(key, nil, hd.PubKeyType(""))
+				res = append(res, info)
+				continue
 			}
 
 			if len(rawInfo.Data) == 0 {
-				return nil, sdkerrors.Wrap(sdkerrors.ErrKeyNotFound, key)
+				fmt.Println(sdkerrors.Wrap(sdkerrors.ErrKeyNotFound, key))
+
+				// add the name of the key in case the user wants to retrieve it
+				// afterwards
+				info := newOfflineInfo(key, nil, hd.PubKeyType(""))
+				res = append(res, info)
+				continue
 			}
 
 			info, err := unmarshalInfo(rawInfo.Data)
 			if err != nil {
-				return nil, err
+				fmt.Printf("err for key %s: %q\n", key, err)
+
+				// add the name of the key in case the user wants to retrieve it
+				// afterwards
+				info = newOfflineInfo(key, nil, hd.PubKeyType(""))
 			}
 
 			res = append(res, info)