Merge pull request #348 from recognizegroup/feature/TD-596_add_output…

…_with_correct_name_for_connection_string TD-596 - Add output with correct name for primary_connection_string
recognizegroup · Aug 14, 2023 · 96cb8e4 · 96cb8e4
2 parents 3c281e9 + da54e54
commit 96cb8e4
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/modules/azure/storage_account_private/outputs.tf b/modules/azure/storage_account_private/outputs.tf
@@ -15,7 +15,15 @@ output "primary_access_key" {
   sensitive = true
 }
 
-output "primary_access_connection_string" {
+output "primary_connection_string" {
   value     = azurerm_storage_account.storage_account.primary_connection_string
   sensitive = true
 }
+
+# Deprecated in favor of primary_connection_string
+# TODO: Remove deprecated output in next major version
+output "primary_access_connection_string" {
+  description = "Deprecated in favor of primary_connection_string"
+  value       = azurerm_storage_account.storage_account.primary_connection_string
+  sensitive   = true
+}
diff --git a/modules/azure/storage_account_public/outputs.tf b/modules/azure/storage_account_public/outputs.tf
@@ -15,7 +15,15 @@ output "primary_access_key" {
   sensitive = true
 }
 
-output "primary_access_connection_string" {
+output "primary_connection_string" {
   value     = azurerm_storage_account.storage_account.primary_connection_string
   sensitive = true
 }
+
+# Deprecated in favor of primary_connection_string
+# TODO: Remove deprecated output in next major version
+output "primary_access_connection_string" {
+  description = "Deprecated in favor of primary_connection_string"
+  value       = azurerm_storage_account.storage_account.primary_connection_string
+  sensitive   = true
+}