Skip to content

Commit f5e89bd

Browse files
committed
feat: add support for ignore_vulnerability_alerts_during_read
1 parent bca455a commit f5e89bd

File tree

5 files changed

+18
-4
lines changed

5 files changed

+18
-4
lines changed

Diff for: CHANGELOG.md

+2-1
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
1212
- BREAKING: update to provider `~> 4.31` for supporting default squash and merge commit titles and messages
1313

1414
### Added
15+
1516
- Add support for `squash_merge_commit_title`
1617
- Add support for `squash_merge_commit_message`
1718
- Add support for `merge_commit_title`
1819
- Add support for `merge_commit_message`
19-
20+
- Add support for `ignore_vulnerability_alerts_during_read`
2021

2122
## [0.17.0]
2223

Diff for: README.md

+4
Original file line numberDiff line numberDiff line change
@@ -330,6 +330,10 @@ See [variables.tf] and [examples/] for details and use-cases.
330330
Set to `false` to disable security alerts for vulnerable dependencies.
331331
Enabling requires alerts to be enabled on the owner level.
332332

333+
- [**`ignore_vulnerability_alerts_during_read`**](#var-ignore_vulnerability_alerts_during_read): *(Optional `bool`)*<a name="var-ignore_vulnerability_alerts_during_read"></a>
334+
335+
Set to `true` to not call the vulnerability alerts endpoint so the resource can also be used without admin permissions during read.
336+
333337
- [**`archive_on_destroy`**](#var-archive_on_destroy): *(Optional `bool`)*<a name="var-archive_on_destroy"></a>
334338

335339
Set to `false` to not archive the repository instead of deleting on destroy.

Diff for: README.tfdoc.hcl

+7
Original file line numberDiff line numberDiff line change
@@ -409,6 +409,13 @@ section {
409409
END
410410
}
411411

412+
variable "ignore_vulnerability_alerts_during_read" {
413+
type = bool
414+
description = <<-END
415+
Set to `true` to not call the vulnerability alerts endpoint so the resource can also be used without admin permissions during read.
416+
END
417+
}
418+
412419
variable "archive_on_destroy" {
413420
type = bool
414421
default = true

Diff for: main.tf

+3-2
Original file line numberDiff line numberDiff line change
@@ -11,14 +11,14 @@ locals {
1111
visibility = var.visibility == null ? lookup(var.defaults, "visibility", local.private_visibility) : var.visibility
1212
has_issues = var.has_issues == null ? lookup(var.defaults, "has_issues", false) : var.has_issues
1313
has_projects = var.has_projects == null ? lookup(var.defaults, "has_projects", false) : length(var.projects) > 0 ? true : var.has_projects
14+
has_downloads = var.has_downloads == null ? lookup(var.defaults, "has_downloads", false) : var.has_downloads
1415
has_wiki = var.has_wiki == null ? lookup(var.defaults, "has_wiki", false) : var.has_wiki
1516
allow_merge_commit = var.allow_merge_commit == null ? lookup(var.defaults, "allow_merge_commit", true) : var.allow_merge_commit
1617
allow_rebase_merge = var.allow_rebase_merge == null ? lookup(var.defaults, "allow_rebase_merge", false) : var.allow_rebase_merge
1718
allow_squash_merge = var.allow_squash_merge == null ? lookup(var.defaults, "allow_squash_merge", false) : var.allow_squash_merge
1819
allow_auto_merge = var.allow_auto_merge == null ? lookup(var.defaults, "allow_auto_merge", false) : var.allow_auto_merge
1920
delete_branch_on_merge = var.delete_branch_on_merge == null ? lookup(var.defaults, "delete_branch_on_merge", true) : var.delete_branch_on_merge
2021
is_template = var.is_template == null ? lookup(var.defaults, "is_template", false) : var.is_template
21-
has_downloads = var.has_downloads == null ? lookup(var.defaults, "has_downloads", false) : var.has_downloads
2222
auto_init = var.auto_init == null ? lookup(var.defaults, "auto_init", true) : var.auto_init
2323
gitignore_template = var.gitignore_template == null ? lookup(var.defaults, "gitignore_template", "") : var.gitignore_template
2424
license_template = var.license_template == null ? lookup(var.defaults, "license_template", "") : var.license_template
@@ -42,7 +42,8 @@ locals {
4242

4343
issue_labels_merge_with_github_labels = local.gh_labels
4444
# Per default, GitHub activates vulnerability alerts for public repositories and disables it for private repositories
45-
vulnerability_alerts = var.vulnerability_alerts != null ? var.vulnerability_alerts : local.private ? false : true
45+
vulnerability_alerts = var.vulnerability_alerts != null ? var.vulnerability_alerts : local.private ? false : true
46+
ignore_vulnerability_alerts_during_read = var.ignore_vulnerability_alerts_during_read == null ? try(var.defaults.ignore_vulnerability_alerts_during_read, null) : var.ignore_vulnerability_alerts_during_read
4647
}
4748

4849
locals {

Diff for: variables.tf

+2-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ variable "defaults" {
4646
# gitignore_template = "terraform"
4747
# license_template = "mit"
4848
# default_branch = "main"
49-
# branch_protections_v3 = []
5049
# topics = ["topic-1", "topic-2"]
5150
# template = "terraform-module-template"
5251
# vulnerability_alerts = true
@@ -56,6 +55,8 @@ variable "defaults" {
5655
# squash_merge_commit_message = "COMMIT_MESSAGES"
5756
# merge_commit_title = "PR_TITLE"
5857
# merge_commit_message = "PR_BODY"
58+
# vulnerability_alerts = true
59+
# ignore_vulnerability_alerts_during_read = true
5960
# }
6061

6162
default = {}

0 commit comments

Comments
 (0)