-
Notifications
You must be signed in to change notification settings - Fork 156
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Invalid ARN when creating aws.wafv2.WebAclLoggingConfiguration #1214
Comments
Just want to add that aws.kinesis.FirehoseDeliveryStream("aws-waf-logs-modelbuilderStream") was created in us-east-1 for waf logging as the waf is assigned to CloudFront and the name is prefixed with 'aws-waf-logs-' as aws recommends (https://docs.aws.amazon.com/waf/latest/developerguide/logging.html) |
you need to replace "global" in the arn string to "regional" |
Can't do that as that's the ARN for for webacl. Since the webacl is assigned to a CloudFront distribution you can have only global scope waf. |
@emiioan Could you take a look at hashicorp/terraform-provider-aws#13955 (comment) and see if this helps? |
Hello, saw that closed issue before opening mine. I'm afraid that there is no explanation there regarding the fix. The only notable thing was regarding kinesis stream name which needs to start with aws-waf-logs-* and mine does start with this prefix. |
Hello, can you please advise if there are any updates on this? I implemented kinesis delivery stream in other projects with pulumi and still doing this step manually. |
I haven't had the capacity to look into this issue, unfortunately. If you run |
Thanks for reply, I used the debug you provided and my guess is that aws-sdk-go it's not parsing/validating ARN format properly:
|
Hi @emiioan - I had this same issue but when using Terraform, which led me here, and as it turns out the issue was in between the chair and the keyboard, as I forgot to set the region to us-east-1 for this resource creation, hence the ARN validation error :) Hopefully this helps! |
@ricoli I am experiencing the same. It is failing even with |
ah no sorry, my problem was the missing alias configuration |
@ricoli Yep, I can confirm another issue between the chair and the keyboard :)) . I works by mentioning the us-east-1 region as you advised. |
You need to ensure your delivery stream's name begins with |
@rererecursive, firstly thanks! Curious where in the docs it states the delivery stream's name must begin with |
@rererecursive Thanks! I've been tearing my hair out over this all day. For anyone else looking, this is mentioned on https://docs.aws.amazon.com/waf/latest/developerguide/logging.html
|
This was fixed by doing the following:
Closing issue. |
Even I was facing the same issue with TF, dropped by here. Tried adding region under provider section and this worked. |
Not only for Kinesis logging, but it goes the same for CloudWatch logging. const logGroup:aws_logs.LogGroup = new aws_logs.LogGroup(this, `logGroupWaf`, {
retention: 30,
logGroupName: `aws-waf-logs-abc`
}); |
@fusion27 the Cloudformation return value for ARN of a Cloudwatch Log Group includes the * on the end. You'll need to use the name instead and build an ARN string |
I got this error:
I fix this issue, for CloudWatch logging by adding at the beginning, in the name of the
|
What an absolute scam |
It's been 4 years and this problem still hasn't been fixed 😒 |
@maksymyaremko can you open a new ticket with repro steps? We look at incoming tickets with repro steps but we're out of capacity to all the closed tickets. |
When trying to create aws.wafv2.WebAclLoggingConfiguration it fails with The ARN isn't valid. A valid ARN begins with arn: and includes other information separated by colons or slashes.
Thing is that the arn format its ok, spoke with AWS support and they said that all the setup was made properly for aws and api calls so it seems to be a pulumi issue.
Here is the example code from @pulumi/aws module:
And this is my code:
When I run Pulumi, I have the following error:
For now I had to manually enable logging to my webacl v2 but It will be nice to have it all done from Pulumi.
Please advise if its something I do wrong or is there any other constructor that I can use to configure logging. I tried to add the ARN as string but same outcome.
Thanks
The text was updated successfully, but these errors were encountered: