From 9768ea10c636916642ef2ff2f0186e740563f0dd Mon Sep 17 00:00:00 2001
From: Joseph Page <josephpage@users.noreply.github.com>
Date: Fri, 18 Aug 2023 14:58:29 +0200
Subject: [PATCH] fix: rename sticky_sessions variable

---
 app.tf             | 4 ++--
 domains.tf         | 2 +-
 outputs.tf         | 2 +-
 scm_integration.tf | 4 ----
 variables.tf       | 8 ++++----
 5 files changed, 8 insertions(+), 12 deletions(-)

diff --git a/app.tf b/app.tf
index 7cbba62..a092a0c 100644
--- a/app.tf
+++ b/app.tf
@@ -10,8 +10,8 @@ resource "scalingo_app" "app" {
   stack_id    = data.scalingo_stack.stack.id
   force_https = !var.authorize_unsecure_http
 
-  sticky_sessions = var.sticky_sessions
-  router_logs     = var.router_logs
+  sticky_session = var.sticky_session
+  router_logs    = var.router_logs
 }
 
 resource "scalingo_container_type" "containers" {
diff --git a/domains.tf b/domains.tf
index 52e7bc3..2b3f9de 100644
--- a/domains.tf
+++ b/domains.tf
@@ -5,7 +5,7 @@ resource "scalingo_domain" "canonical_domain" {
   canonical   = true
   common_name = var.domain
 
-  # Attributes not supported by the provider on v2.0 :
+  # Attributes not supported by the provider on v2.2 :
   # tlscert - optional: SSL Certificate you want to associate with the domain
   # tlskey - optional: Private key used to create the SSL certificate
 }
diff --git a/outputs.tf b/outputs.tf
index bb51fb5..8619ab8 100644
--- a/outputs.tf
+++ b/outputs.tf
@@ -7,7 +7,7 @@ output "app_id" {
 output "all_environment_variables" {
   description = "All environment variables of the Scalingo application (ones added by the terraform module and ones added by Scalingo add-ons)."
   value       = scalingo_app.app.all_environment
-  sensitive = true # Environment variables can contain sensitive information
+  sensitive   = true # Environment variables can contain sensitive information
 }
 
 output "url" {
diff --git a/scm_integration.tf b/scm_integration.tf
index 9470a97..2c12ac0 100644
--- a/scm_integration.tf
+++ b/scm_integration.tf
@@ -21,8 +21,6 @@ data "scalingo_scm_integration" "scm_integration" {
 resource "scalingo_scm_repo_link" "scm_repo_link" {
   for_each = local.scm_integration
 
-  # Due to a bug in the provider, we need to use the app id instead of the app name
-  # Link to the issue : https://github.com/Scalingo/terraform-provider-scalingo/issues/153
   app = scalingo_app.app.id
 
   source = each.value.repo_url
@@ -41,6 +39,4 @@ resource "scalingo_scm_repo_link" "scm_repo_link" {
   delete_stale_enabled                  = var.review_apps.delete_stale_enabled
   hours_before_delete_stale             = var.review_apps.hours_before_delete_stale
   automatic_creation_from_forks_allowed = var.review_apps.automatic_creation_from_forks_allowed
-
-  
 }
diff --git a/variables.tf b/variables.tf
index 17707ec..16605c2 100644
--- a/variables.tf
+++ b/variables.tf
@@ -30,21 +30,21 @@ variable "containers" {
 }
 
 variable "authorize_unsecure_http" {
-  description = "When true, Scalingo does not automatically redirect HTTP traffic to HTTPS"
+  description = "When true, Scalingo does not automatically redirect HTTP traffic to HTTPS. The default behavior is to redirect HTTP traffic to HTTPS (when value is `false`)"
   type        = bool
   default     = false
   nullable    = false
 }
 
 variable "router_logs" {
-  description = "When true, the router logs are included in the application logs."
+  description = "When true, the router logs are included in the application logs. (default: `false`)"
   type        = bool
   default     = false
   nullable    = false
 }
 
-variable "sticky_sessions" {
-  description = "When true, sticky sessions are enabled."
+variable "sticky_session" {
+  description = "When true, sticky sessions are enabled. (default: `false`)"
   type        = bool
   default     = false
   nullable    = false