fix: always allow providers

Author: 2403905

pkg/ocm/provider/authorizer/json/json.go

@@ -173,10 +173,12 @@ func (a *authorizer) IsProviderAllowed(ctx context.Context, pi *ocmprovider.Prov
 	}
 
 	switch {
-	case !providerAuthorized:
-		return errtypes.NotFound(pi.GetDomain())
 	case !a.conf.VerifyRequestHostname:
+		log.Info().Msg("VerifyRequestHostname is disabled. any provider is allowed")
 		return nil
+	case !providerAuthorized:
+		log.Info().Msg("providerAuthorized is false")
+		return errtypes.NotFound(pi.GetDomain())
 	case len(pi.Services) == 0:
 		return ErrNoIP
 	}

pkg/ocm/provider/authorizer/json/json_test.go

@@ -49,7 +49,8 @@ func TestAuthorizer_IsProviderAllowed(t *testing.T) {
 				providerInfo: &ocmprovider.ProviderInfo{
 					Domain: "some.unknown.domain",
 				},
-				expectedError: errtypes.NotFound("some.unknown.domain"),
+				verifyRequestHostname: true,
+				expectedError:         errtypes.NotFound("some.unknown.domain"),
 			},
 			"authorized without host name verification": {
 				providerInfo: &ocmprovider.ProviderInfo{
@@ -65,7 +66,8 @@ func TestAuthorizer_IsProviderAllowed(t *testing.T) {
 				providerInfo: &ocmprovider.ProviderInfo{
 					Domain: "server-two",
 				},
-				expectedError: error(errtypes.NotFound("server-two")),
+				verifyRequestHostname: true,
+				expectedError:         error(errtypes.NotFound("server-two")),
 			},
 		} {
 			t.Run(name, func(t *testing.T) {