Samin061 opened a new pull request, #69585:
URL: https://github.com/apache/airflow/pull/69585
Tasks fetch connections from the API server through `GET
/execution/connections/{conn_id}`. The route used the default string path
converter, so a `conn_id` that contains `/` (for example
`dev-env/project-name`) was split into separate path segments and never
matched, and the task failed with a 404 even though the connection existed.
`airflow connections get` reads connections directly and was unaffected, so the
CLI and Task SDK disagreed for the same connection.
This switches the route to the `:path` converter so slashed connection IDs
route to the handler, mirroring how the variables route
(`/{variable_key:path}`) already handles slashed keys. Added a regression test
for a connection whose ID contains a slash.
closes: #57864
---
##### Was generative AI tooling used to co-author this PR?
- [ ] Yes (please specify the tool below)
--
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]
