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 2020/02/20 05:59:33 UTC

[GitHub] [cloudstack] rhtyd opened a new issue #3897: Review all list APIs and make then honour listall parameter

rhtyd opened a new issue #3897: Review all list APIs and make then honour listall parameter
URL: https://github.com/apache/cloudstack/issues/3897
 
 
   There are many list APIs that should ideally honour the `listall` parameter and when set to true should return all the items a caller/account is authorised to see. Due to a historic use of projectid=-1, most APIs do return all project only resources.
   
   With PR https://github.com/apache/cloudstack/pull/3894 the behaviour has been addressed for some APIs such as listRouters etc where all resources are returned for root admin when both listall=true and projectid=-1 is passed. A review of all list APIs and fix to make the behaviour consistent is asked.
   
   ##### ISSUE TYPE
    * Bug Report
    * Improvement Request
   
   ##### COMPONENT NAME
   <!--
   Categorize the issue, e.g. API, VR, VPN, UI, etc.
   -->
   ~~~
   API
   ~~~
   
   ##### CLOUDSTACK VERSION
   <!--
   New line separated list of affected versions, commit ID for issues on master branch.
   -->
   
   ~~~
   4.13, 4.14+
   ~~~

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


With regards,
Apache Git Services

[GitHub] [cloudstack] weizhouapache commented on issue #3897: Review all list APIs and make then honour listall parameter

Posted by GitBox <gi...@apache.org>.
weizhouapache commented on issue #3897: Review all list APIs and make then honour listall parameter
URL: https://github.com/apache/cloudstack/issues/3897#issuecomment-602001860
 
 
   @rhtyd Should this be fixed before 4.14.0.0 release ?

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


With regards,
Apache Git Services

[GitHub] [cloudstack] rhtyd commented on issue #3897: Review all list APIs and make then honour listall parameter

Posted by GitBox <gi...@apache.org>.
rhtyd commented on issue #3897: Review all list APIs and make then honour listall parameter
URL: https://github.com/apache/cloudstack/issues/3897#issuecomment-602024904
 
 
   @weizhouapache I'll need to check if others APIs need to be supported, the changes were done solely to support infrastructure (VR etc) list views in Primate.

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


With regards,
Apache Git Services