Skip to content

Commit 62f5fab

Browse files
security fixes
1 parent ec61bbb commit 62f5fab

File tree

3 files changed

+5
-14
lines changed

3 files changed

+5
-14
lines changed

README.md

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,17 +13,14 @@ This module helps you create a dynamoDB table. It includes the following:
1313
* autoscaling settings
1414

1515
## Sample Usage
16-
This module requires 2 arguments.
17-
* `cluster_name` simply the name of the cluster
18-
* `log_group` log group name
16+
This module requires a wide set of arguments. Please have a look at variables.tf
1917

2018
```
2119
module "terraform-aws-dynamodb-table" {
2220
source = "git@github.com:elastic2ls/terraform-aws-dynamodb-table.git"
2321
24-
cluster_name = "fargate-ecs"
25-
log_group = "fargate-ecs-log-group"
26-
}
22+
name = elastic2ls-dynamodb
23+
...
2724
```
2825

2926
The module can also be found in the Terraform Registry https://registry.terraform.io/modules/elastic2ls-com/dynamodb-table/aws/latest.

dynamodb.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ resource "aws_dynamodb_table" "table" {
1616
}
1717

1818
point_in_time_recovery {
19-
enabled = var.point_in_time_recovery_enabled
19+
enabled = true
2020
}
2121

2222
dynamic "attribute" {
@@ -99,7 +99,7 @@ resource "aws_dynamodb_table" "table_autoscaled" {
9999
}
100100

101101
point_in_time_recovery {
102-
enabled = var.point_in_time_recovery_enabled
102+
enabled = true
103103
}
104104

105105
dynamic "attribute" {

variables.tf

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,6 @@ variable "read_capacity" {
5050
default = null
5151
}
5252

53-
variable "point_in_time_recovery_enabled" {
54-
description = "Enable point-in-time recovery"
55-
type = bool
56-
default = false
57-
}
58-
5953
variable "ttl_enabled" {
6054
description = "Enalbe TTL"
6155
type = bool

0 commit comments

Comments
 (0)