auth
7:edf7ffe90517 Browse Files
Handle fillTokens errors. Remove TODOs surrounding errors when saving tokens.
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