Skip to content

Sanitize URLs before logging them

Ahmad Sherif requested to merge fix/credentials-leakage into master

Closes #1052

Benchmarking data:

BenchmarkUrlSanitizerWithoutSanitization-8        100000             12918 ns/op
BenchmarkUrlSanitizerWithSanitization-8            50000             34616 ns/op

which is not that good, but given we're only performing sanitization for a few methods I think we can accept the hit.

Edited by 🤖 GitLab Bot 🤖

Merge request reports