Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Set log message to debug #1970

Merged
merged 1 commit into from
Oct 28, 2021
Merged

Set log message to debug #1970

merged 1 commit into from
Oct 28, 2021

Conversation

dloucasfx
Copy link
Contributor

With this config

monitors:
  - type: http
    host: localhost
    port: 4443
    useHTTPS: true
    skipVerify: true

and if the server is using invalid cert, we'll log this info message which can pollute the log file.

failed verify certificate       {"kind": "receiver", "name": "smartagent/http", "monitorType": "http", "url": "https://localhost:4443/", "error": "x509: certificate is not valid for any names, but wanted to match localhost"}

instead, move it to debug
Signed-off-by: Dani Louca dlouca@splunk.com

Signed-off-by: Dani Louca <dlouca@splunk.com>
@dloucasfx dloucasfx merged commit bdc4672 into main Oct 28, 2021
@dloucasfx dloucasfx deleted the SWAT-3899 branch October 28, 2021 16:19
@AchimGrolimund
Copy link

I think that by setting the tag skipVerify: true
this message should no longer come because you skip it, so no more testing should take place.
Am i wrong?

Greetings Achim

@dloucasfx
Copy link
Contributor Author

@AchimGrolimund
skipVerify is used by the http client when it checks the server status getHTTPStats.
The error addressed here is part of the monitor cert validation getTLSStats , which is independent of skipVerify

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants