henry3260 commented on PR #58344:
URL: https://github.com/apache/airflow/pull/58344#issuecomment-3566461934
> Nice!
>
> It would be nice to add `pytest.mark.parametrize` with slash value key in
`task-sdk-integration-tests/tests/task_sdk_tests/test_xcom_operations.py` tests.
>
> Thanks!
Thanks for reviewing!
Slash keys aren’t supported today: the route pattern
/{dag_id}/{run_id}/{task_id}/{key} treats {key} as a single path segment, so
"folder/sub/value" splits into multiple segments and won’t match. Adding
support means restructuring the routes (e.g. {key:path} plus reworking
/item/{offset} and /slice), or switching to a query param (?key=). That’s a
broader change, so I propose we keep the simple key test in this PR, optionally
mark a slash-key test as xfail, and follow up with a dedicated issue/PR for
proper support. If you prefer to fold the refactor into this PR, let me
know—just want to confirm scope first.
--
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]
