fixed error messages to give more context (#1196)

* fixed error messages to give more context

* updated CHANGELOG.md

CHANGELOG.md

@@ -1,6 +1,7 @@
 # Changelog
 
 ## HEAD
+- `bnsd`: updated error messages to better reflect the account module spec
 - `bnsd`: account targets are cleared when an account is transferred
 - `bnsd`: domain renew expiration time equals to domain expiration time + configuration domain renew duration, if this time is before current block time then renew time becomes current block time + configuration domain renew duration
 - `bnsd`: revert burn feature

cmd/bnsd/x/account/handler.go

@@ -740,7 +740,7 @@ func (h *deleteAccountHandler) validate(ctx weave.Context, db weave.KVStore, tx
 		authenticated = h.auth.HasAddress(ctx, domain.Admin)
 	}
 	if !authenticated {
-		return nil, errors.Wrap(errors.ErrUnauthorized, "only owner can delete an account")
+		return nil, errors.Wrap(errors.ErrUnauthorized, "only account owner or domain owner (if domain has a superuser) can delete an account")
 	}
 	return &msg, nil
 }
@@ -851,7 +851,7 @@ func (h *addAccountCertificateHandler) validate(ctx weave.Context, db weave.KVSt
 
 	var account Account
 	if err := h.accounts.One(db, accountKey(msg.Name, msg.Domain), &account); err != nil {
-		return nil, nil, errors.Wrap(err, "account")
+		return nil, nil, errors.Wrapf(err, "account %s does not exist", msg.Name)
 	}
 	if weave.IsExpired(ctx, account.ValidUntil) {
 		return nil, nil, errors.Wrap(errors.ErrExpired, "account")