Fixed condition in product check

ezimuel · ezimuel · commit 5a15755b9273 · 2022-12-27T15:40:20.000+01:00
diff --git a/lib/Search/Elasticsearch/Role/Cxn.pm b/lib/Search/Elasticsearch/Role/Cxn.pm
@@ -343,7 +343,7 @@ sub process_response {
     
     # Product check
     if ( $code >= 200 and $code < 300 ) {
-        unless ($headers->{'X-Elastic-Product'} eq 'Elasticsearch' ) {
+        if (!defined $headers->{'X-Elastic-Product'} || $headers->{'X-Elastic-Product'} eq 'Elasticsearch') {
             throw( "ProductCheck", "The client noticed that the server is not Elasticsearch and we do not support this unknown product" );
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -343,7 +343,7 @@ sub process_response {`
`343`	`343`
`344`	`344`	`# Product check`
`345`	`345`	`if ( $code >= 200 and $code < 300 ) {`
`346`		`- unless ($headers->{'X-Elastic-Product'} eq 'Elasticsearch' ) {`
	`346`	`+ if (!defined $headers->{'X-Elastic-Product'} \|\| $headers->{'X-Elastic-Product'} eq 'Elasticsearch') {`
`347`	`347`	`throw( "ProductCheck", "The client noticed that the server is not Elasticsearch and we do not support this unknown product" );`
`348`	`348`	`}`
`349`	`349`	`}`