beenull/crowdsec
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

fix #919 : display error message (#929)

Browse source 
* fix #919

* fix tests
This commit is contained in:
Thibault "bui" Koechlin 2021-09-02 12:46:32 +02:00 committed by GitHub
parent ea640dfb6d
commit bed90a832e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
pkg/apiclient/auth.go
View file

@ -149,7 +149,11 @@ func (t *JWTTransport) refreshJwtToken() error {
defer resp.Body.Close()
if resp.StatusCode < 200 || resp.StatusCode >= 300 {
return fmt.Errorf("received response status %q when fetching %v", resp.Status, req.URL)
log.Debugf("received response status %q when fetching %v", resp.Status, req.URL)
err = CheckResponse(resp)
if err != nil {
return err
}
}
if err := json.NewDecoder(resp.Body).Decode(&response); err != nil {

2
pkg/apiclient/auth_service_test.go
View file

@ -88,7 +88,7 @@ func TestWatcherAuth(t *testing.T) {
MachineID: &mycfg.MachineID,
Password: &mycfg.Password,
})
assert.Contains(t, err.Error(), "403 Forbidden")
assert.Contains(t, err.Error(), "API error: access forbidden")
}

2
pkg/apiclient/client_test.go
View file

@ -106,7 +106,7 @@ func TestNewClientKo(t *testing.T) {
})
_, _, err = client.Alerts.List(context.Background(), AlertsListOpts{})
assert.Contains(t, err.Error(), `received response status "401 Unauthorized"`)
assert.Contains(t, err.Error(), `API error: bad login/password`)
log.Printf("err-> %s", err)
}