You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/03/06 08:09:38 UTC

[GitHub] [airflow] vemikhaylov commented on issue #14635: UpdateMask is useless for POST/PATCH if readOnly=true in the openapi specs

vemikhaylov commented on issue #14635:
URL: https://github.com/apache/airflow/issues/14635#issuecomment-791893272


   If I understand it correctly, the problem is that specification states passing a DAG object as the body of the PATCH request is allowed. However, there's the `dag_id` field there, which is `readOnly` and when we pass a full `DAG` object to the `PATCH`, it fails with the `400` because of the `dag_id`.
   
   Maybe all the fields except `dag_id` should be moved to the separate entity (e.g. `DAGData` or something), while `DAG` should include it and extend with `dag_id` (using `allOf`). Then the `PATCH` method will use the `DAGData` schema, not the `DAG` one.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org