fix(externalIpProvider/plain): when reading response body (#54)

Fixing an error where the service crashes reading the response body, when no content-length header was provided in the response.

Reviewed-on: #54
Reviewed-by: branch-buddy <branch-buddy@t00n.de>
Co-authored-by: Timo Behrendt <t.behrendt@t00n.de>
Co-committed-by: Timo Behrendt <t.behrendt@t00n.de>

pkg/externalIpProvider/plain/plain.go

@@ -2,6 +2,7 @@ package externalIpProvider
 
 import (
 	"errors"
+	"io"
 	"net"
 	"net/http"
 	"net/url"
@@ -40,12 +41,15 @@ func (p *ExternalIpProviderImplPlain) GetExternalIp() (net.IP, error) {
 	}
 
 	if res.StatusCode != 200 {
+		res.Body.Close()
 		return nil, errors.New("unexpected status code")
 	}
 
-	responseBody := make([]byte, res.ContentLength)
+	responseBody, err := io.ReadAll(res.Body)
-	res.Body.Read(responseBody)
+	res.Body.Close()
-	defer res.Body.Close()
+	if err != nil {
+		return nil, err
+	}
 
 	parsedIp := net.ParseIP(string(responseBody))
 	if parsedIp == nil {