[alexanto]

makes the API clients' error handling library-neutral, so a transport built on any HTTP library behaves the same: any exception from transport.execute() is a failed request and it gets logged, and the client keeps running

because of this a malformed header no longer crashes anything, so I added a validate headers check

[Copilot]

🟡 Changes recommended

Header validation incorrectly permits None for app_name, instance_id, and connection_id, bypassing the documented constructor TypeError and deferring handling to the HTTP library.

1 open issue

• Validate required header fields even when their values are None · New

What changed in this PR

This PR makes API request error handling independent of the HTTP library, aligning the synchronous and asynchronous clients.

Changes:

• Treat transport exceptions during registration and metrics submission as failed requests while preserving payload serialization errors.
• Validate header values during configuration and remove redundant initialization-time URL checks.
• Update regression tests and migration documentation for the revised behavior.

File	Description
v7_MIGRATION_GUIDE.md	Documents header validation and trailing-newline handling.
UnleashClient/​utils.py	Adds header-value validation.
UnleashClient/​config.py	Validates configured header values at construction.
UnleashClient/​clients/​unleash_client.py	Removes redundant URL validation during initialization.
UnleashClient/​clients/​async_unleash_client.py	Removes redundant URL validation and outdated exception documentation.
UnleashClient/​_unleash_api_client.py	Handles transport exceptions without requests-specific exception types.
UnleashClient/​_async_unleash_api_client.py	Broadens transport error handling while preserving cancellation.
tests/​unit_tests/​test_utils.py	Tests header validation and secret-safe errors.
tests/​unit_tests/​test_unleash_api_client.py	Tests transport failures and serialization-error propagation.
tests/​unit_tests/​test_config.py	Tests constructor header validation.
tests/​unit_tests/​test_async_unleash_api_client.py	Tests transport failures, closed transports, and serialization errors.
CHANGELOG.md	Records validation and registration-error behavior changes.

_{Review effort: Balanced}

[alexanto]

this is hardly a regression and there are defaults for those values in case of None, so I dont this this makes much sense