Alwaysgaurav1 opened a new pull request, #69439:
URL: https://github.com/apache/airflow/pull/69439

   ### Description
   
   The Execution API endpoint `GET /execution/connections/{connection_id}` uses 
a standard path parameter `{connection_id}`, which causes FastAPI to treat 
slashes in connection IDs as path separators. This means connection IDs 
containing slashes (e.g. `my/slashed/conn`) return a 404 Not Found instead of 
resolving correctly.
   
   This is inconsistent with the Variables endpoint which already uses 
`{variable_key:path}` to support keys with slashes.
   
   This PR fixes the issue by changing the route definition from 
`/{connection_id}` to `/{connection_id:path}`, allowing connection IDs with 
slashes to be properly resolved.
   
   A new unit test `test_connection_get_with_slash` is added to verify that 
connections with slashed IDs can be fetched successfully.
   
   Closes: #57864


-- 
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]

Reply via email to