Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Don't put remote certificate errors into the log #5684

@jo-so

Description

@jo-so

Description

I'm having tons of warnings in the log which aren't useful

2019-07-14 17:41:18,489 - synapse.http.matrixfederationclient - 482 - WARNING - PUT-61673- {GET-O-1430} [matrix.sebdu.de] Request failed: GET matrix://matrix.sebdu.de/_matrix/key/v2/server/ed25519%3Aa_vZnw: ResponseNeverReceived:[Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')])]
2019-07-14 17:41:18,489 - synapse.crypto.keyring - 792 - WARNING - PUT-61673- Error looking up keys {'ed25519:a_vZnw': 1563118872464} from matrix.sebdu.de: Failed to connect to remote server
2019-07-14 17:41:18,490 - synapse.federation.transport.server - 286 - WARNING - PUT-61673- authenticate_request failed: 401: No key for matrix.sebdu.de with ids in ['ed25519:a_vZnw'] (min_validity 1563118872464)

To find failed requests, the log message with 'certificate verify failed' is enough. These warnings aren't needed.

Proposed patch: 0001-federation-client-Don-t-put-remote-certificate-error.patch.txt

Version information

  • Version: 1.0.0-2
  • Install method: apt install matrix-synapse
  • Platform: Debian/stable

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-LoggingSynapse's logs (structured or otherwise). Not metrics.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions