Edit: Internal changes for ofl-s pending, do not merge.
Changelog:
[x] Fixes issue where disabling client auth with TLS-enabled, broke taskrunner experiments (#1123).
[x] Renames tls -> use_tls in the plan.
[x] Renames disable_client_auth -> require_client_auth in the plan. NOTE: This change has been made to reduce cognitive overhead of knowing how to enable mTLS in communication between the client-server.
Before this PR: (disable_client_auth=False, tls=True) meant enabling mTLS.
After this PR: (require_client_auth=True, use_tls=True) will enable mTLS.
Default behaviour FL plans is to use mTLS.
[x] Migrates logger.warn -> logger.warning to hide deprecation warnings
Edit: Internal changes for ofl-s pending, do not merge.
Changelog:
tls -> use_tls
in the plan.disable_client_auth -> require_client_auth
in the plan. NOTE: This change has been made to reduce cognitive overhead of knowing how to enable mTLS in communication between the client-server.disable_client_auth=False, tls=True
) meant enabling mTLS.require_client_auth=True, use_tls=True
) will enable mTLS.logger.warn -> logger.warning
to hide deprecation warningsTested with 4 combinations of two updated flags.