[GitHub] [openwhisk] ningyougang removed a comment on pull request #4058: Add protect feature to avoid update or delete actions by mistake

[GitBox <gi...@apache.org>]

ningyougang removed a comment on pull request #4058:
URL: https://github.com/apache/openwhisk/pull/4058#issuecomment-850162422


   After read you guys review suggestion, 2 important points
   
   * For the permission data storage, we can depend on permission data in DB rather than relying on action fields.
   
   Actually i don't know how to design this(e.g. how to storage and where to storage) and make it more better.
   
   * one of my thought is that there is no need to add permissions controls for action's "owner" since he/she can anyway update/remove/invoke his/her action
   
   It seems just need to control the permission for the shared user. (e.g. the shared user whether has permission to download code, execute the codes)
   
   But for this pr, our initial target is `avoid the owner update or delete actions by mistake`, it seems if don't add permission to action's owner, can't reach the initial target


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