Change IsAllowedEngine func to return true when packageEnforcementEnabled is false (AST-76112)

.github/workflows/pr-linter.yml

@@ -23,7 +23,7 @@ jobs:
             exit 1
           fi
 
-          if ! [[ "$PR_BRANCH" =~ ^(bug|feature|other)/ ]]; then
+          if ! [[ "$PR_BRANCH" =~ ^(bug|fix|feature|other)/ ]]; then
             echo "::error::Branch name must start with 'bug/' or 'feature/' or 'other/'."
             exit 1
           fi

CODEOWNERS

@@ -2,4 +2,4 @@
 # Each line is a file pattern followed by one or more owners
 
 # Specify the default owners for the entire repository
-*       @OrShamirCM @AlvoBen
+*       @AlvoBen @greensd4 @miryamfoiferCX

internal/services/asca_test.go

@@ -5,6 +5,7 @@ import (
 	"testing"
 
 	errorconstants "github.com/checkmarx/ast-cli/internal/constants/errors"
+	"github.com/checkmarx/ast-cli/internal/wrappers"
 	"github.com/checkmarx/ast-cli/internal/wrappers/grpcs"
 	"github.com/checkmarx/ast-cli/internal/wrappers/mock"
 	"github.com/stretchr/testify/assert"
@@ -130,3 +131,20 @@ func TestCreateASCAScanRequest_EngineRunningAndDefaultAgentAndNoLicense_Success(
 	assert.Nil(t, wrapperParams.ASCAWrapper.HealthCheck())
 	_ = wrapperParams.ASCAWrapper.ShutDown()
 }
+
+func TestCreateASCAScanRequest_CheckLicenseWhenPackageEnforcementFFOff_SuccessWithNilError(t *testing.T) {
+	port, err := getAvailablePort()
+	if err != nil {
+		t.Fatalf("Failed to get available port: %v", err)
+	}
+
+	mock.Flag = wrappers.FeatureFlagResponseModel{Name: wrappers.PackageEnforcementEnabled, Status: false}
+
+	wrapperParams := AscaWrappersParam{
+		JwtWrapper:          wrappers.NewJwtWrapper(),
+		FeatureFlagsWrapper: &mock.FeatureFlagsMockWrapper{},
+		ASCAWrapper:         grpcs.NewASCAGrpcWrapper(port),
+	}
+	err = checkLicense(false, wrapperParams)
+	assert.Nil(t, err)
+}

internal/wrappers/jwt-helper.go

@@ -78,8 +78,9 @@ func (*JWTStruct) IsAllowedEngine(engine string, featureFlagsWrapper FeatureFlag
 				return true, nil
 			}
 		}
+		return false, nil
 	}
-	return false, nil
+	return true, nil
 }
 
 func prepareEngines(engines []string) map[string]bool {