SQS Queue Redrive Policy typing

[kg-jcaron]

Hello!

• Vote on this issue by adding a 👍 reaction
• If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

The SQS redrivePolicy is not strictly typed (uses any in typescript, object in dotnet). Everything seems to work correctly with it, but it would be nice to have strict typing for the property names (deadLetterTargetArn and maxReceiveCount) like in the aws classic provider if possible.

[mjeffryes]

Thanks for pointing this out @kg-jcaron it should be possible to generate more useful types here.

[danielrbradley]

This appears to be an issue with the upstream specifications.

The type of this field is currently specified as: https://github.com/pulumi/pulumi-aws-native/blob/550eb8c0c6559a06d584fc996e2ab58f7af97d69/aws-cloudformation-schema/aws-sqs-queue.json#L83-L86

However in the CloudFormation documentation it more accurately describes the type as having two fields:

{ "deadLetterTargetArn" : String, "maxReceiveCount" : Integer }

This also applies to RedriveAllowPolicy.

[mikhailshilkov]

@danielrbradley Have you reported this in https://github.com/aws-cloudformation/cloudformation-coverage-roadmap/? (that's the upstream repo we should report CF issues to)

[danielrbradley]

No, I've not written this up as an issue for them yet. For some providers there's also separate repositories in https://github.com/aws-cloudformation/ for their specific implementations which are worth checking too for issues.

[mikhailshilkov]

Let's report the upstream issue as the first step

[danielrbradley]

Reported: https://github.com/aws-cloudformation/cloudformation-coverage-roadmap/issues/2012