You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Prachi Damle <Pr...@citrix.com> on 2014/04/30 21:00:40 UTC

[ACS44]cherry pick CLOUDSTACK-6517

Hi Hugo/Daan,

Can you please cherry-pick following to 4.4:

Commit hash:    e14c2ec7241ec2333ea3ac3d56a0e938fe1bf131

CLOUDSTACK-6517: IAM - Admin is allowed to create PortFowarding rule for a regular user, when admin does not have " UseEntry" permission for IpAddress.
Contained in branches: 4.4-forward

Thanks,
Prachi

RE: [ACS44]cherry pick CLOUDSTACK-6517

Posted by Prachi Damle <Pr...@citrix.com>.
Yes, I will check my work environment again for this.

Prachi
-----Original Message-----
From: Daan Hoogland [mailto:daan.hoogland@gmail.com] 
Sent: Wednesday, April 30, 2014 11:37 PM
To: dev
Subject: Re: [ACS44]cherry pick CLOUDSTACK-6517

On Wed, Apr 30, 2014 at 9:00 PM, Prachi Damle <Pr...@citrix.com> wrote:
> e14c2ec7241ec2333ea3ac3d56a0e938fe1bf131


Prachi, your commit contains dos line endings:
-        IAMPolicyPermissionVO permit =
_policyPermissionDao.findByPolicyAndEntity(iamPolicyId, entityType, scope, scopeId, action, perm);
+        IAMPolicyPermissionVO permit =
_policyPermissionDao.findByPolicyAndEntity(iamPolicyId, entityType, scope,^M
+                scopeId, action, perm, accessType);^M

Since I only saw them in java files I will let it pass but please adjust your work environment to prevent the line endings to enter the system. When they end up in configuration files or scripts things might break inexplicably.

--
Daan

Re: [ACS44]cherry pick CLOUDSTACK-6517

Posted by Daan Hoogland <da...@gmail.com>.
On Wed, Apr 30, 2014 at 9:00 PM, Prachi Damle <Pr...@citrix.com> wrote:
> e14c2ec7241ec2333ea3ac3d56a0e938fe1bf131


Prachi, your commit contains dos line endings:
-        IAMPolicyPermissionVO permit =
_policyPermissionDao.findByPolicyAndEntity(iamPolicyId, entityType,
scope, scopeId, action, perm);
+        IAMPolicyPermissionVO permit =
_policyPermissionDao.findByPolicyAndEntity(iamPolicyId, entityType,
scope,^M
+                scopeId, action, perm, accessType);^M

Since I only saw them in java files I will let it pass but please
adjust your work environment to prevent the line endings to enter the
system. When they end up in configuration files or scripts things
might break inexplicably.

-- 
Daan