YoannAbriel commented on PR #63359: URL: https://github.com/apache/airflow/pull/63359#issuecomment-4042083374
Thanks for the pointer to #59564 — you're right, the previous approach was wrong. Reworked: instead of adding PATCH to `ExtendedResourceMethod`, this now widens `_MAP_METHOD_NAME_TO_FAB_ACTION_NAME` from `dict[ExtendedResourceMethod, str]` to `dict[str, str]` and adds `"PATCH": ACTION_CAN_EDIT`. No enum changes, no route changes — just the map fix so PATCH resolves to `can_edit` like PUT does. This follows the approach you described in https://github.com/apache/airflow/pull/59564#pullrequestreview-2704428170: accept `str` keys and map PATCH → PUT action, without extending the auth model. -- 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]
