diff --git a/controller/model/edge_router_manager.go b/controller/model/edge_router_manager.go
index c8fe499c7..cc50f465f 100644
--- a/controller/model/edge_router_manager.go
+++ b/controller/model/edge_router_manager.go
@@ -227,7 +227,7 @@ func (self *EdgeRouterManager) IsAccessToEdgeRouterAllowed(identityId, serviceId
 		return nil
 	})
 	if err != nil {
-		return false, nil
+		return false, err
 	}
 	return result, nil
 }
@@ -249,7 +249,7 @@ func (self *EdgeRouterManager) IsSharedEdgeRouterPresent(identityId, serviceId s
 		return nil
 	})
 	if err != nil {
-		return false, nil
+		return false, err
 	}
 	return result, nil
 }
diff --git a/router/xgress_edge/dialer.go b/router/xgress_edge/dialer.go
index 193e1efbe..091ff60df 100644
--- a/router/xgress_edge/dialer.go
+++ b/router/xgress_edge/dialer.go
@@ -35,7 +35,7 @@ type dialer struct {
 	options *Options
 }
 
-func (dialer dialer) IsTerminatorValid(id string, destination string) bool {
+func (dialer *dialer) IsTerminatorValid(id string, destination string) bool {
 	destParts := strings.Split(destination, ":")
 	if len(destParts) != 2 {
 		return false
diff --git a/router/xgress_edge_tunnel/dialer.go b/router/xgress_edge_tunnel/dialer.go
index ef13a4499..56288c62b 100644
--- a/router/xgress_edge_tunnel/dialer.go
+++ b/router/xgress_edge_tunnel/dialer.go
@@ -61,7 +61,7 @@ func (self *tunneler) Dial(params xgress.DialParams) (xt.PeerData, error) {
 		return nil, err
 	}
 
-	log.Infof("successful connection %v->%v for destination %v", conn.LocalAddr(), conn.RemoteAddr(), destination)
+	log.Debugf("successful connection %v->%v for destination %v", conn.LocalAddr(), conn.RemoteAddr(), destination)
 
 	xgConn := xgress_common.NewXgressConn(conn, halfClose, false)
 	peerData := make(xt.PeerData, 3)