auth

Paddy 2014-08-13 Parent:3423c552e249 Child:fb2fd59f9930

7:edf7ffe90517 Browse Files

Handle fillTokens errors. Remove TODOs surrounding errors when saving tokens.

access.go

     1.1 --- a/access.go	Wed Aug 13 05:53:50 2014 -0400
     1.2 +++ b/access.go	Wed Aug 13 06:07:10 2014 -0400
     1.3 @@ -142,7 +142,7 @@
     1.4  
     1.5  	err = fillTokens(&data, true, ctx)
     1.6  	if err != nil {
     1.7 -		// TODO: return error
     1.8 +		ctx.RenderJSONError(w, ErrorServerError, "Internal server error.", ctx.Config.DocumentationDomain)
     1.9  		return
    1.10  	}
    1.11  	ctx.RenderJSONToken(w, data)
    1.12 @@ -203,7 +203,7 @@
    1.13  	}
    1.14  	err = fillTokens(&data, true, ctx)
    1.15  	if err != nil {
    1.16 -		// TODO: return error
    1.17 +		ctx.RenderJSONError(w, ErrorServerError, "Internal server error.", ctx.Config.DocumentationDomain)
    1.18  		return
    1.19  	}
    1.20  	ctx.RenderJSONToken(w, data)
    1.21 @@ -253,7 +253,7 @@
    1.22  
    1.23  	err = fillTokens(&data, true, ctx)
    1.24  	if err != nil {
    1.25 -		// TODO: return error
    1.26 +		ctx.RenderJSONError(w, ErrorServerError, "Internal server error.", ctx.Config.DocumentationDomain)
    1.27  		return
    1.28  	}
    1.29  	ctx.RenderJSONToken(w, data)
    1.30 @@ -289,7 +289,7 @@
    1.31  
    1.32  	err = fillTokens(&data, true, ctx)
    1.33  	if err != nil {
    1.34 -		// TODO: return error
    1.35 +		ctx.RenderJSONError(w, ErrorServerError, "Internal server error.", ctx.Config.DocumentationDomain)
    1.36  		return
    1.37  	}
    1.38  	ctx.RenderJSONToken(w, data)
    1.39 @@ -307,8 +307,8 @@
    1.40  	// save access token
    1.41  	err = ctx.Tokens.SaveAccess(*data)
    1.42  	if err != nil {
    1.43 -		// TODO: abstract out error
    1.44 -		return err
    1.45 +		// TODO: log error
    1.46 +		return InternalServerError
    1.47  	}
    1.48  
    1.49  	// remove authorization token