This message was deleted.
# generals
sparse-intern-71089
08/14/2019, 8:17 PMThis message was deleted.
b
boundless-coat-60947
08/14/2019, 8:18 PMnable to compile TypeScript:
s3/index.ts(228,9): error TS2322: Type ‘{ “NotIPAddress”: { “awsSourceIp” string[]; }; }’ is not assignable to type ‘Conditions | Promise<Conditions> | OutputInstance<Conditions> | undefined’.
Type ‘{ “NotIPAddress”: { “awsSourceIp” string[]; }; }’ is not assignable to type ‘Conditions’.
Property ‘“NotIPAddress”’ is incompatible with index signature.
Type ‘{ “awsSourceIp” string[]; }’ is not assignable to type ‘ConditionArguments’.
Property ‘“aws:SourceIp”’ is incompatible with index signature.
Type ‘string[]’ is not assignable to type ‘Input<string>’.
Type ‘string[]’ is not assignable to type ‘string’.
boundless-coat-60947
08/14/2019, 8:18 PMIs this a bug, or am I just not understanding this interface?
boundless-coat-60947
08/14/2019, 8:19 PMhttps://docs.aws.amazon.com/IAM/latest/UserGuide/reference_policies_elements_condition_operators.html#Conditions_IPAddress seems to indicate you should be able to pass in multiple values to aws:SourceIp
boundless-coat-60947
08/14/2019, 8:45 PMLooks like this https://github.com/pulumi/pulumi-aws/blob/master/sdk/nodejs/iam/documents.ts#L115 is limiting the ConditionArguments to a single String Value
l
lemon-spoon-91807
08/14/2019, 10:18 PMLooks like a good catch @boundless-coat-60947
lemon-spoon-91807
08/14/2019, 10:19 PMWould you be willing to file a bug on this over at github.com/pulumi/aws? If not, i can do it for you!
b
boundless-coat-60947
08/15/2019, 3:46 PMYep - I’ll get it filed. Thanks for giving it another set of eyes.
boundless-coat-60947
08/15/2019, 4:32 PMhttps://github.com/pulumi/pulumi-aws/issues/705 - how’s that look?
2 Views