Add ability to allow list group namespaces to bypass ECA
3 unresolved threads
3 unresolved threads
Compare changes
+ 4
− 0
@@ -22,6 +22,7 @@ ALLOW_LIST_PROJECTS = ['/eclipse/oniro-core/meta-ts',
@@ -22,6 +22,7 @@ ALLOW_LIST_PROJECTS = ['/eclipse/oniro-core/meta-ts',
@@ -118,6 +119,9 @@ if (nil_or_empty(project_url)) then
@@ -118,6 +119,9 @@ if (nil_or_empty(project_url)) then
elsif (ALLOW_LIST_PROJECTS.any? {|repo_namespace| "#{HOST_URL}#{repo_namespace}" == project_url}) then
More of an observation but we define projects with a slash at the start of the string but not groups?
It's due to how that check is done, as it uses a different method to check it which is less efficient. I didn't want to touch it since it's working, but this check should be a bit more robust as it doesn't rely on the host URL