Skip to content

Commit

Permalink
fix: rename sticky_sessions variable
Browse files Browse the repository at this point in the history
  • Loading branch information
josephpage committed Aug 22, 2023
1 parent 2d9f4bd commit 9768ea1
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 12 deletions.
4 changes: 2 additions & 2 deletions app.tf
Original file line number Diff line number Diff line change
Expand Up @@ -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" {
Expand Down
2 changes: 1 addition & 1 deletion domains.tf
Original file line number Diff line number Diff line change
Expand Up @@ -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
}
Expand Down
2 changes: 1 addition & 1 deletion outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -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" {
Expand Down
4 changes: 0 additions & 4 deletions scm_integration.tf
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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


}
8 changes: 4 additions & 4 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 9768ea1

Please sign in to comment.