vincbeck commented on PR #47138: URL: https://github.com/apache/airflow/pull/47138#issuecomment-2688285298
Why do we need both? What is the different between `PUT` and `PATCH`? Maybe using HTTP terms in auth manager was not that much of a good idea actually but here we just want to express action, what kind of action the user is trying to do? - Create (POST) - Edit (PUT) - Delete (DELETE) - Read (GET) If you want to add PATCH just because you have PATCH routes defined in the fastapi application I think this is wrong. We should then have a layer that translate fastapi method to auth manager action and convert PATCH to PUT in that layer -- 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]
