fix(pstorage): add colon for better error formating (#292)

fix file mode for bbolt file

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **Bug Fixes**
- Improved error handling to prevent unnecessary error messages when
saving existing account data.
- Enhanced error messages for account and certificate operations to
provide clearer information.

- **Chores**
  - Updated file permissions for database files to ensure secure access.
- Improved internal error logging for better visibility of account query
issues.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Signed-off-by: Artur Troian <[email protected]>
Co-authored-by: Artur Troian <[email protected]>

Author: troian

cmd/provider-services/cmd/certs.go

@@ -311,7 +311,7 @@ func (c *accountQuerier) run() error {
 			}
 		case acc := <-accountRespCh:
 			err := pstorage.AddAccount(c.ctx, acc.GetAddress(), acc.GetPubKey())
-			if err != nil {
+			if err != nil && !errors.Is(err, pconfig.ErrExists) {
 				c.log.Error("unable to save account pubkey into storage", "owner", acc.GetAddress().String(), "err", err)
 			}
 		case req := <-c.peerCertCh:
@@ -466,12 +466,14 @@ func (c *accountQuerier) accountQuerier(reqch <-chan sdk.Address, respch chan<-
 
 			res, err := c.qc.Query().Auth().Account(c.ctx, &authtypes.QueryAccountRequest{Address: addr.String()})
 			if err != nil {
+				c.log.Error("fetching account info", "err", err.Error(), "account", addr.String())
 				requests = append(requests, addr)
 				continue
 			}
 
 			var acc authtypes.AccountI
 			if err := cctx.InterfaceRegistry.UnpackAny(res.Account, &acc); err != nil {
+				c.log.Error("unpacking account info", "err", err.Error(), "account", addr.String())
 				requests = append(requests, addr)
 				continue
 			}

tools/pconfig/bbolt/bbolt.go

@@ -9,7 +9,6 @@ import (
 	"errors"
 	"fmt"
 	"math/big"
-	"os"
 
 	ctypes "github.com/akash-network/akash-api/go/node/cert/v1beta3"
 	"github.com/cosmos/cosmos-sdk/crypto/keys/secp256k1"
@@ -50,7 +49,7 @@ type account struct {
 }
 
 func NewBBolt(path string) (pconfig.Storage, error) {
-	db, err := bbolt.Open(path, os.ModeType, nil)
+	db, err := bbolt.Open(path, 0600, nil)
 	if err != nil {
 		return nil, err
 	}

tools/pconfig/pconfig.go

@@ -16,10 +16,10 @@ var (
 	ErrInvalidArgs          = errors.New("pstorage: invalid arguments")
 	ErrNotExists            = errors.New("pstorage: not exists")
 	ErrExists               = errors.New("pstorage: exists")
-	ErrAccountNotExists     = fmt.Errorf("%w account", ErrNotExists)
-	ErrAccountExists        = fmt.Errorf("%w account", ErrExists)
-	ErrCertificateNotExists = fmt.Errorf("%w certificate", ErrNotExists)
-	ErrCertificateExists    = fmt.Errorf("%w certificate", ErrExists)
+	ErrAccountNotExists     = fmt.Errorf("%w: account", ErrNotExists)
+	ErrAccountExists        = fmt.Errorf("%w: account", ErrExists)
+	ErrCertificateNotExists = fmt.Errorf("%w: certificate", ErrNotExists)
+	ErrCertificateExists    = fmt.Errorf("%w: certificate", ErrExists)
 )
 
 type StorageR interface {