feat: GitlabRepositoryProvider now handles leading slashes for content URLs

Author: Nathan Johnson

modules/nextflow/src/main/groovy/nextflow/scm/GitlabRepositoryProvider.groovy

@@ -88,6 +88,7 @@ class GitlabRepositoryProvider extends RepositoryProvider {
         //  https://docs.gitlab.com/ee/api/repository_files.html#get-raw-file-from-repository
         //
         final ref = revision ?: getDefaultBranch()
+        path = path.replaceAll("^/+", "")  // leading slashes cause 400s with GitLab
         final encodedPath = URLEncoder.encode(path,'utf-8')
         return "${config.endpoint}/api/v4/projects/${getProjectName()}/repository/files/${encodedPath}?ref=${ref}"
     }

modules/nextflow/src/test/groovy/nextflow/scm/GitlabRepositoryProviderTest.groovy

@@ -19,6 +19,7 @@ package nextflow.scm
 import spock.lang.IgnoreIf
 import spock.lang.Requires
 import spock.lang.Specification
+import spock.lang.Unroll
 
 /**
  *
@@ -50,16 +51,20 @@ class GitlabRepositoryProviderTest extends Specification {
     }
 
     @Requires({System.getenv('NXF_GITLAB_ACCESS_TOKEN')})
-    def 'should read file content'() {
+    @Unroll
+    def 'should read file content: #contentURL'() {
         given:
         def token = System.getenv('NXF_GITLAB_ACCESS_TOKEN')
         def config = new ProviderConfig('gitlab').setAuth(token)
 
         when:
         def repo = new GitlabRepositoryProvider('pditommaso/hello', config)
-        def result = repo.readText('main.nf')
+        def result = repo.readText(contentURL)
         then:
         result.trim().startsWith('#!/usr/bin/env nextflow')
+        where:
+        contentURL << ['main.nf', '/main.nf']
+
     }
 
     @Requires({System.getenv('NXF_GITLAB_ACCESS_TOKEN')})