Review feedback

api/grpcserver/post_service.go

@@ -75,7 +75,7 @@ func (s *PostService) connectionAllowed() bool {
 // requests to the PoST node and receive responses.
 func (s *PostService) Register(stream pb.PostService_RegisterServer) error {
 	if !s.connectionAllowed() {
-		return status.Error(codes.PermissionDenied, "connection not allowed: node has no coinbase set in config")
+		return status.Error(codes.FailedPrecondition, "connection not allowed: node has no coinbase set in config")
 	}
 
 	err := stream.SendMsg(&pb.NodeRequest{

api/grpcserver/post_service_test.go

@@ -362,6 +362,6 @@ func Test_PostService_Connection_NotAllowed(t *testing.T) {
 
 	_, err = stream.Recv()
 	require.Error(t, err)
-	require.Equal(t, codes.PermissionDenied, status.Code(err))
+	require.Equal(t, codes.FailedPrecondition, status.Code(err))
 	require.ErrorContains(t, err, "connection not allowed")
 }

node/node.go

@@ -1436,7 +1436,11 @@ func (app *App) grpcService(svc grpcserver.Service, lg log.Log) (grpcserver.Serv
 		return service, nil
 	case grpcserver.Post:
 		service := grpcserver.NewPostService(app.addLogger(PostServiceLogger, lg).Zap())
-		service.AllowConnections(app.Config.SMESHING.CoinbaseAccount != "")
+		isCoinbaseSet := app.Config.SMESHING.CoinbaseAccount != ""
+		if !isCoinbaseSet {
+			lg.Warning("coinbase account is not set, connections from remote post services will be rejected")
+		}
+		service.AllowConnections(isCoinbaseSet)
 		app.grpcServices[svc] = service
 		return service, nil
 	case grpcserver.PostInfo: