Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-28393

Allow case insensitive matching on companyName/repoName in GH plugin

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Major Major
    • github-plugin
    • None
    • Jenkins 1.613
      GitHub 1.11.3

      The github jenkins documentation at https://wiki.jenkins-ci.org/display/JENKINS/GitHub+Plugin says:

      ```
      When creating a job, specify that is connects to git. Under "Github project", put in: git@github.com:Person/Project.git Under "Source Code Management" select Git, and put in git@github.com:Person/Project.git
      ```

      When we try git@github.com:<company>/<project>.git it does not work. However, https://github.com/<company>/<project> works for us.

      This causes a problem with push notifications, though. I get the following messages in the logs:

      ```
      May 13, 2015 4:05:26 PM FINE com.cloudbees.jenkins.GitHubWebHook
      Considering to poke webserver_test_master
      May 13, 2015 4:05:26 PM FINE com.cloudbees.jenkins.GitHubWebHook
      Skipped webserver_test_master because it doesn't have a matching repository.
      ```

      Presumably there are no names found in the addRepositories method in https://github.com/jenkinsci/github-plugin/blob/master/src/main/java/com/cloudbees/jenkins/GitHubRepositoryNameContributor.java

      Incidentally, it would be very helpful to have a logger in that function which details what URLs it is parsing and what names it gets (if any).

      Thanks for the very useful tool!

          [JENKINS-28393] Allow case insensitive matching on companyName/repoName in GH plugin

          AngledLuffa created issue -
          Kirill Merkushev made changes -
          Summary Original: Repository URL "https://github.com/&lt;group&gt;/&lt;repo>" doesn't work New: Allow case insensitive matching on companyName/repoName in GH plugin
          Kanstantsin Shautsou made changes -
          Assignee New: Kirill Merkushev [ lanwen ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 163232 ] New: JNJira + In-Review [ 181145 ]

            lanwen Kirill Merkushev
            angledluffa AngledLuffa
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: