tirkarthi commented on PR #54336: URL: https://github.com/apache/airflow/pull/54336#issuecomment-3177612035
`navigate` doesn't seem to encode uri by default. There is configuration to validate `run_id` by regex through `allowed_run_id_pattern`. Since the links to dag run in the runs tab and dag_run list page are not encoded I guess it's okay to skip encoding. Encoded urls also work as per my testing. Current URL : http://localhost:8000/dags/example_display_name/runs/manual__2025-08-12T03:51:19.882418+00:00 Encoded URL : http://localhost:8000/dags/example_display_name/runs/manual__2025-08-12T03%3A51%3A19.882418%2B00%3A00 https://github.com/remix-run/react-router/blob/a089339c58ee0758db81b0a9c680b308628dddf7/packages/react-router/lib/hooks.tsx#L362 https://airflow.apache.org/docs/apache-airflow/stable/configurations-ref.html#allowed-run-id-pattern -- 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]
