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

Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2021/11/30 06:59:34 UTC

[GitHub] [cloudstack] weizhouapache commented on pull request #5729: server: fix non-root users are able to list system networks by id

weizhouapache commented on pull request #5729:
URL: https://github.com/apache/cloudstack/pull/5729#issuecomment-982340555


   > @weizhouapache 'by id' means it should be uuid, right?
   
   @sureshanaparti 
   I removed "by id" in the title. 
   
   the points of this PR are.
   (1) domain admin and normal users should not be able to list system networks (by id=200, 201...). cloudstack networks starts with id=200 in DB, the first networks are system networks (Public/Management/Control)
   
   (2) domain admin should not be able to list system networks.


-- 
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: commits-unsubscribe@cloudstack.apache.org

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