GaetanJuvin opened a new pull request, #58122:
URL: https://github.com/apache/airflow/pull/58122
## Summary
- Fix Execution API connection lookups by accepting
`/{connection_id:path}`, decoding the path segment, and returning a 422 with
`reason=missing_conn_type` when a secret/env var omits `conn_type`;
updated docs to explain the requirement.
- Percent-encode connection IDs on the Task SDK client side so IDs
containing `/` resolve correctly, keeping server/client behavior in sync.
- Added regression tests for both sides plus a newsfragment; happy to take
feedback if a different routing/validation strategy is preferred (this version
leans on URI encode/decode).
closes: #57864
## Testing
- `pytest
airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_connections.py`
- `pytest
task-sdk/tests/task_sdk/api/test_client.py::TestConnectionOperations`
- `prek --all-files` (minus Breeze-only hooks on this machine)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
