> + .fromPort(startPort)
> + .toPort(endPort)
> + .build(),
> + groupId);
> + }
> + }
> +
> + return getSecurityGroupById(ZoneAndId.fromZoneAndId(zone,
> id).slashEncode());
> + }
> +
> + @Override
> + public SecurityGroup removeIpPermission(IpPermission ipPermission,
> SecurityGroup group) {
> + String zone = group.getLocation().getId();
> + String id = group.getId();
> +
> + Optional<? extends SecurityGroupApi> sgApi =
> client.getSecurityGroupExtensionForZone(zone);
No null check for `zone`?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/144/files#r6344684
