uranusjr commented on a change in pull request #16609:
URL: https://github.com/apache/airflow/pull/16609#discussion_r657107518
##########
File path: airflow/api_connexion/endpoints/user_endpoint.py
##########
@@ -62,3 +64,43 @@ def get_users(limit, order_by='id', offset=None):
users = query.offset(offset).limit(limit).all()
return user_collection_schema.dump(UserCollection(users=users,
total_entries=total_entries))
+
+
[email protected]_access([(permissions.ACTION_CAN_CREATE,
permissions.RESOURCE_USER)])
+def post_user():
+ """Create a new user"""
Review comment:
Thinking about this again, I wonder whether a 5xx would make sense,
since a 4xx indicates the client can fix something to make the request work,
but no amount of client-side changes can make this error go away, it is only
fixable at the server side (and once it’s resolved, the client can re-send the
request unchanged, which is not typical of 4xx either).
Maybe something like 501 Not Implemented?
--
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:
[email protected]
