Fix healthCheckEndpoint when using docker containers to not miss the …

…scheme in the URL
ba-st · Feb 17, 2022 · 8eaa730 · 8eaa730
1 parent a6f218b
commit 8eaa730
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 17 deletions.
diff --git a/source/Stargate-Consul/ConsulAwareStargateApplication.class.st b/source/Stargate-Consul/ConsulAwareStargateApplication.class.st
@@ -10,14 +10,21 @@ ConsulAwareStargateApplication class >> isAbstract [
 	^ self = ConsulAwareStargateApplication
 ]
 
-{ #category : #'*Stargate-Consul' }
+{ #category : #private }
 ConsulAwareStargateApplication class >> stargateConfigurationParameters [
 
-	^ super stargateConfigurationParameters , { MandatoryConfigurationParameter
-		    named: 'Consul Agent Location'
-		    describedBy: 'Location of the Consul Agent. Leave empty to disable the plugin'
-		    inside: self sectionsForStargateConfiguration
-		    convertingWith: #asUrl }
+	^ super stargateConfigurationParameters , { 
+		  (MandatoryConfigurationParameter
+			   named: 'Consul Agent Location'
+			   describedBy:
+			   'Location of the Consul Agent. Leave empty to disable the plugin'
+			   inside: self sectionsForStargateConfiguration
+			   convertingWith: #asUrl).
+		  (OptionalConfigurationParameter
+			   named: 'Scheme'
+			   describedBy: 'Transport scheme. It''s used to configure Consul HTTP checks'
+			   inside: self sectionsForStargateConfiguration
+			   defaultingTo: 'http') }
 ]
 
 { #category : #'private - building' }
@@ -91,6 +98,7 @@ ConsulAwareStargateApplication >> healthCheckEndpoint [
 	baseUrl := self
 		           withDockerHostnameDo: [ :hostname | 
 			           ZnUrl new
+				           scheme: self stargateConfiguration scheme;
 				           host: hostname;
 				           port: self stargateConfiguration port;
 				           yourself

diff --git a/source/Stargate-Consul/ConsulAwareStargateApplication.extension.st b/source/Stargate-Consul/ConsulAwareStargateApplication.extension.st