scriptoriumadmin/featurebase
1
0
Fork 0
mirror of https://github.com/featurebasedb/featurebase.git synced 2026-08-28 10:54:59 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

add http status check to authenticate

Browse source
This commit is contained in:
Samir Patel 2022-04-07 16:03:27 -05:00
parent eea6a40fe0
commit bddccf6ead
2 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
authn/authenticate.go
View file

@ -129,6 +129,9 @@ func (a *Auth) Authenticate(ctx context.Context, bearer string) (*UserInfo, erro
if err != nil {
return nil, errors.Wrap(err, "refreshing token")
}
if resp.StatusCode != http.StatusOK {
return nil, fmt.Errorf("refreshing token: %s", resp.Status)
}
defer resp.Body.Close()
var t oauth2.Token
if err := json.NewDecoder(resp.Body).Decode(&t); err != nil {

2
authn/authenticate_internal_test.go
View file

@ -198,7 +198,7 @@ func TestAuthenticate(t *testing.T) {
refresh: true,
errOnRefresh: true,
exp: -17764800,
err: fmt.Errorf("decoding refreshed token: invalid character 'b' looking for beginning of value"),
err: fmt.Errorf("refreshing token: 500 Internal Server Error"),
},
}
for _, test := range cases {