diff --git a/auth/authenticator.go b/auth/authenticator.go index 14f881cde..1640a842e 100644 --- a/auth/authenticator.go +++ b/auth/authenticator.go @@ -17,12 +17,11 @@ package auth import ( "fmt" - "log" "os" - "github.com/vmware/harbor/models" + "github.com/vmware/harbor/utils/log" - "github.com/astaxie/beego" + "github.com/vmware/harbor/models" ) // Authenticator provides interface to authenticate user credentials. @@ -37,7 +36,7 @@ var registry = make(map[string]Authenticator) // Register add different authenticators to registry map. func Register(name string, authenticator Authenticator) { if _, dup := registry[name]; dup { - log.Printf("authenticator: %s has been registered", name) + log.Infof("authenticator: %s has been registered", name) return } registry[name] = authenticator @@ -50,7 +49,7 @@ func Login(m models.AuthModel) (*models.User, error) { if authMode == "" || m.Principal == "admin" { authMode = "db_auth" } - beego.Debug("Current AUTH_MODE is ", authMode) + log.Debug("Current AUTH_MODE is ", authMode) authenticator, ok := registry[authMode] if !ok { diff --git a/auth/ldap/ldap.go b/auth/ldap/ldap.go index bc24cb8fe..53d17a174 100644 --- a/auth/ldap/ldap.go +++ b/auth/ldap/ldap.go @@ -18,15 +18,15 @@ package ldap import ( "errors" "fmt" - "log" "os" "strings" + "github.com/vmware/harbor/utils/log" + "github.com/vmware/harbor/auth" "github.com/vmware/harbor/dao" "github.com/vmware/harbor/models" - "github.com/astaxie/beego" "github.com/mqu/openldap" ) @@ -44,7 +44,7 @@ func (l *Auth) Authenticate(m models.AuthModel) (*models.User, error) { if ldapURL == "" { return nil, errors.New("Can not get any available LDAP_URL.") } - beego.Debug("ldapURL:", ldapURL) + log.Debug("ldapURL:", ldapURL) p := m.Principal for _, c := range metaChars { @@ -66,7 +66,7 @@ func (l *Auth) Authenticate(m models.AuthModel) (*models.User, error) { } baseDn := fmt.Sprintf(ldapBaseDn, m.Principal) - beego.Debug("baseDn:", baseDn) + log.Debug("baseDn:", baseDn) err = ldap.Bind(baseDn, m.Password) if err != nil { @@ -83,7 +83,7 @@ func (l *Auth) Authenticate(m models.AuthModel) (*models.User, error) { return nil, err } if len(result.Entries()) != 1 { - log.Printf("Found more than one entry.") + log.Warningf("Found more than one entry.") return nil, nil } en := result.Entries()[0] @@ -100,7 +100,7 @@ func (l *Auth) Authenticate(m models.AuthModel) (*models.User, error) { } } - beego.Debug("username:", u.Username, ",email:", u.Email, ",realname:", u.Realname) + log.Debug("username:", u.Username, ",email:", u.Email, ",realname:", u.Realname) exist, err := dao.UserExists(u, "username") if err != nil {