Merge branch 'hotfix/2.7.1'

deviceinsight · Nov 3, 2020 · ce2aa29 · ce2aa29
2 parents 546c80d + 319a5c0
commit ce2aa29
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.adoc b/CHANGELOG.adoc
@@ -1,5 +1,9 @@
 = Changelog
 
+== Version 2.7.1
+
+* Fix inverted status check which _fails_ for any 2xx HTTP status
+
 == Version 2.7.0
 
 * Accept any 2xx HTTP status code when deploying Helm charts

diff --git a/pom.xml b/pom.xml
@@ -4,7 +4,7 @@
 
 	<groupId>com.deviceinsight.helm</groupId>
 	<artifactId>helm-maven-plugin</artifactId>
-	<version>2.7.0</version>
+	<version>2.7.1</version>
 	<packaging>maven-plugin</packaging>
 
 	<name>Helm Maven Plugin</name>

diff --git a/src/main/kotlin/com/deviceinsight/helm/DeployMojo.kt b/src/main/kotlin/com/deviceinsight/helm/DeployMojo.kt
@@ -167,7 +167,7 @@ class DeployMojo : AbstractMojo() {
 
 			httpClient.execute(request).use { response ->
 				val statusCode = response.statusLine.statusCode
-				if ( (statusCode >= 200) && (statusCode < 300) ) {
+				if (statusCode < 200 || statusCode >= 300) {
 					throw RuntimeException("Unexpected status code when executing $chartPublishMethod request to " +
 						"chart repo ${chartDeploymentRequest.chartPublishUrl()}: $statusCode")
 				}