databaseauth: Return ErrNotAuthorized instead of generic error.

octavore · octavore · commit 9092927cf7a5 · 2018-01-23T05:51:12.000-08:00
diff --git a/users/databaseauth/module.go b/users/databaseauth/module.go
@@ -1,7 +1,6 @@
 package databaseauth
 
 import (
-	"errors"
 	"net/http"
 
 	"github.com/octavore/naga/service"
@@ -74,7 +73,7 @@ func (m *Module) handleLogin(rw http.ResponseWriter, req *http.Request, par rout
 		return err
 	}
 	if !valid {
-		return errors.New("invalid user")
+		return router.ErrNotAuthorized
 	}
 	err = m.Sessions.CreateSession(userID, rw)
 	if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,6 @@`
`1`	`1`	`package databaseauth`
`2`	`2`
`3`	`3`	`import (`
`4`		`- "errors"`
`5`	`4`	`"net/http"`
`6`	`5`
`7`	`6`	`"github.com/octavore/naga/service"`
`@@ -74,7 +73,7 @@ func (m Module) handleLogin(rw http.ResponseWriter, req http.Request, par rout`
`74`	`73`	`return err`
`75`	`74`	`}`
`76`	`75`	`if !valid {`
`77`		`- return errors.New("invalid user")`
	`76`	`+ return router.ErrNotAuthorized`
`78`	`77`	`}`
`79`	`78`	`err = m.Sessions.CreateSession(userID, rw)`
`80`	`79`	`if err != nil {`