• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Trivial Trivial
    • core

      As commented by James on the previous PR, the trailing dots are valid in URL.

          [JENKINS-51158] Allow trailing dot in Root URL

          Code changed in jenkins
          User: Wadeck Follonier
          Path:
          core/src/main/java/jenkins/util/UrlHelper.java
          core/src/test/java/jenkins/util/UrlHelperTest.java
          http://jenkins-ci.org/commit/jenkins/c79f3f719c3f320ff6f0b17fae7d15ee2c1cbbcb
          Log:
          JENKINS-51158 Improve Root URL validation

          • remove the bug for ipv6
          • remove the support of query that was not expected
          • add support for trailing dots
          • add lots of unit tests

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Wadeck Follonier Path: core/src/main/java/jenkins/util/UrlHelper.java core/src/test/java/jenkins/util/UrlHelperTest.java http://jenkins-ci.org/commit/jenkins/c79f3f719c3f320ff6f0b17fae7d15ee2c1cbbcb Log: JENKINS-51158 Improve Root URL validation remove the bug for ipv6 remove the support of query that was not expected add support for trailing dots add lots of unit tests

          Code changed in jenkins
          User: Devin Nusbaum
          Path:
          core/src/main/java/jenkins/util/UrlHelper.java
          core/src/test/java/jenkins/util/UrlHelperTest.java
          http://jenkins-ci.org/commit/jenkins/b03beb72f33c13ff57c761fe9a7a6a38fc195729
          Log:
          Merge pull request #3424 from Wadeck/JENINS-51158_IMPROVE_ROOT_URL_VALIDATION

          JENKINS-51158 Improve Root URL validation

          Compare: https://github.com/jenkinsci/jenkins/compare/f33731e7ec7a...b03beb72f33c
          *NOTE:* This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

          Functionality will be removed from GitHub.com on January 31st, 2019.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Devin Nusbaum Path: core/src/main/java/jenkins/util/UrlHelper.java core/src/test/java/jenkins/util/UrlHelperTest.java http://jenkins-ci.org/commit/jenkins/b03beb72f33c13ff57c761fe9a7a6a38fc195729 Log: Merge pull request #3424 from Wadeck/JENINS-51158_IMPROVE_ROOT_URL_VALIDATION JENKINS-51158 Improve Root URL validation Compare: https://github.com/jenkinsci/jenkins/compare/f33731e7ec7a...b03beb72f33c * NOTE: * This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/ Functionality will be removed from GitHub.com on January 31st, 2019.

          Devin Nusbaum added a comment -

          Released in Jenkins 2.124 (rerelease of 2.123). See the changelog.

          Devin Nusbaum added a comment - Released in Jenkins 2.124 (rerelease of 2.123). See the changelog .

          Oleg Nenashev added a comment -

          Needs backporting to 2.121.x IMHO

          Oleg Nenashev added a comment - Needs backporting to 2.121.x IMHO

          Code changed in jenkins
          User: Wadeck Follonier
          Path:
          core/src/main/java/jenkins/util/UrlHelper.java
          core/src/test/java/jenkins/util/UrlHelperTest.java
          http://jenkins-ci.org/commit/jenkins/f0311944a059da0a69c7822311e579918d7c2369
          Log:
          JENKINS-51158 Improve Root URL validation

          • remove the bug for ipv6
          • remove the support of query that was not expected
          • add support for trailing dots
          • add lots of unit tests

          (cherry picked from commit c79f3f719c3f320ff6f0b17fae7d15ee2c1cbbcb)

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Wadeck Follonier Path: core/src/main/java/jenkins/util/UrlHelper.java core/src/test/java/jenkins/util/UrlHelperTest.java http://jenkins-ci.org/commit/jenkins/f0311944a059da0a69c7822311e579918d7c2369 Log: JENKINS-51158 Improve Root URL validation remove the bug for ipv6 remove the support of query that was not expected add support for trailing dots add lots of unit tests (cherry picked from commit c79f3f719c3f320ff6f0b17fae7d15ee2c1cbbcb)

            wfollonier Wadeck Follonier
            wfollonier Wadeck Follonier
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: