> +
> + }
> + }
> +
> + return getSecurityGroupById(ZoneAndId.fromZoneAndId(zone,
> id).slashEncode());
> + }
> +
> + @Override
> + public SecurityGroup removeIpPermission(IpProtocol protocol, int
> startPort, int endPort,
> + Multimap<String, String>
> tenantIdGroupNamePairs,
> + Iterable<String> ipRanges,
> + Iterable<String> groupIds,
> 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#r6344690
