🚨 issue (security): The ServerCertificateValidationByPass method name contains a typo and suggests potentially risky behavior.
Beyond the minor typo (ByPass → Bypass), exposing an API that disables certificate validation is inherently risky. To reduce misuse, consider renaming it to something clearly test-only (e.g., DisableServerCertificateValidationForTesting) and/or limiting it to internal or test-only usage so it’s not available in production contexts.
Originally posted by @sourcery-ai[bot] in #30 (comment)
🚨 issue (security): The
ServerCertificateValidationByPassmethod name contains a typo and suggests potentially risky behavior.Beyond the minor typo (
ByPass→Bypass), exposing an API that disables certificate validation is inherently risky. To reduce misuse, consider renaming it to something clearly test-only (e.g.,DisableServerCertificateValidationForTesting) and/or limiting it to internal or test-only usage so it’s not available in production contexts.Originally posted by @sourcery-ai[bot] in #30 (comment)