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

Check log level before dumping resp (#1243)

Browse source 
* Check log level before dumping resp
* Sleep longer in func tests

Signed-off-by: Shivam Sandbhor <shivam.sandbhor@gmail.com>
This commit is contained in:
Shivam Sandbhor 2022-02-08 20:10:01 +05:30 committed by GitHub
parent 5c7c12c62d
commit 76e3612088
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
pkg/apiclient/client_http.go
View file

@ -84,13 +84,15 @@ func (c *ApiClient) Do(ctx context.Context, req *http.Request, v interface{}) (*
return newResponse(resp), err
}
for k, v := range resp.Header {
log.Debugf("[headers] %s : %s", k, v)
}
if log.GetLevel() >= log.DebugLevel {
for k, v := range resp.Header {
log.Debugf("[headers] %s : %s", k, v)
}
dump, err := httputil.DumpResponse(resp, true)
if err == nil {
log.Debugf("Response: %s", string(dump))
dump, err := httputil.DumpResponse(resp, true)
if err == nil {
log.Debugf("Response: %s", string(dump))
}
}
response := newResponse(resp)

2
scripts/func_tests/tests_post-install_0base.sh
View file

@ -38,7 +38,7 @@ ${CSCLI} version || fail "cannot run cscli version"
## alerts
# alerts list at startup should just return one entry : community pull
sleep 5
sleep 40
${CSCLI} alerts list -ojson | ${JQ} '. | length >= 1' || fail "expected at least one entry from cscli alerts list"
## capi
${CSCLI} capi status || fail "capi status should be ok"