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

When validating the ad set-up with several domains the check fails due a space after the comma

      As showed in the images, there is a false error when checking the ad set-up - when it is working properly.

          [JENKINS-38294] When validating the ad set-up with several domains the check fails due a space after the comma

          Code changed in jenkins
          User: Félix Belzunce Arcos
          Path:
          src/main/java/hudson/plugins/active_directory/ActiveDirectorySecurityRealm.java
          http://jenkins-ci.org/commit/active-directory-plugin/936986143c17f4f8d43a93d8dde21f74d3ca06d8
          Log:
          JENKINS-38294 Trim the domains so a space after comma does not get affected

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Félix Belzunce Arcos Path: src/main/java/hudson/plugins/active_directory/ActiveDirectorySecurityRealm.java http://jenkins-ci.org/commit/active-directory-plugin/936986143c17f4f8d43a93d8dde21f74d3ca06d8 Log: JENKINS-38294 Trim the domains so a space after comma does not get affected

          Code changed in jenkins
          User: Felix Belzunce Arcos
          Path:
          src/main/java/hudson/plugins/active_directory/ActiveDirectorySecurityRealm.java
          http://jenkins-ci.org/commit/active-directory-plugin/3163157ce6a68abbfff89a9e1c2d06ad0435b2aa
          Log:
          Merge pull request #43 from fbelzunc/JENKINS-38294

          [FIXED JENKINS-38294] Trim the domains so a space after comma does not get introduced

          Compare: https://github.com/jenkinsci/active-directory-plugin/compare/d6bb427d070f...3163157ce6a6

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Felix Belzunce Arcos Path: src/main/java/hudson/plugins/active_directory/ActiveDirectorySecurityRealm.java http://jenkins-ci.org/commit/active-directory-plugin/3163157ce6a68abbfff89a9e1c2d06ad0435b2aa Log: Merge pull request #43 from fbelzunc/ JENKINS-38294 [FIXED JENKINS-38294] Trim the domains so a space after comma does not get introduced Compare: https://github.com/jenkinsci/active-directory-plugin/compare/d6bb427d070f...3163157ce6a6

          Code changed in jenkins
          User: Félix Belzunce Arcos
          Path:
          src/main/java/hudson/plugins/active_directory/ActiveDirectorySecurityRealm.java
          http://jenkins-ci.org/commit/active-directory-plugin/c76d69179ba72dad4633e18e7a014f99892ad2c3
          Log:
          JENKINS-38294 Add a warning when displayed name is not used with several domains

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Félix Belzunce Arcos Path: src/main/java/hudson/plugins/active_directory/ActiveDirectorySecurityRealm.java http://jenkins-ci.org/commit/active-directory-plugin/c76d69179ba72dad4633e18e7a014f99892ad2c3 Log: JENKINS-38294 Add a warning when displayed name is not used with several domains

          Code changed in jenkins
          User: Felix Belzunce Arcos
          Path:
          src/main/java/hudson/plugins/active_directory/ActiveDirectorySecurityRealm.java
          http://jenkins-ci.org/commit/active-directory-plugin/50db0a024404daa8189751851cac94c60555d3a6
          Log:
          Merge pull request #44 from fbelzunc/JENKINS-38295

          JENKINS-38294 Add a warning when displayed name is not used with several domains

          Compare: https://github.com/jenkinsci/active-directory-plugin/compare/3163157ce6a6...50db0a024404

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Felix Belzunce Arcos Path: src/main/java/hudson/plugins/active_directory/ActiveDirectorySecurityRealm.java http://jenkins-ci.org/commit/active-directory-plugin/50db0a024404daa8189751851cac94c60555d3a6 Log: Merge pull request #44 from fbelzunc/ JENKINS-38295 JENKINS-38294 Add a warning when displayed name is not used with several domains Compare: https://github.com/jenkinsci/active-directory-plugin/compare/3163157ce6a6...50db0a024404

            fbelzunc Félix Belzunce Arcos
            fbelzunc Félix Belzunce Arcos
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: