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

Setup wizard suggests root URL not ending in slash

      The implementation of JENKINS-31661 suggests http://localhost:8080/jenkins as the root URL if you run with --prefix=/jenkins. It must be http://localhost:8080/jenkins/ in this case, or http://localhost:8080/ if the prefix is undefined (so, blank).

          [JENKINS-51660] Setup wizard suggests root URL not ending in slash

          Jesse Glick added a comment -

          I suppose for settings compatibility, we should append a slash to a saved root URL which is missing one, but the validator should require it going forward.

          Jesse Glick added a comment - I suppose for settings compatibility, we should append a slash to a saved root URL which is missing one, but the validator should require it going forward.

          In the current version, the method that save the URL add the slash if it's missing and in addition, the read method ensure the slash is present.

          I just changed the JavaScript URL guessing method to let the trailing slash.

          Wadeck Follonier added a comment - In the current version, the method that save the URL add the slash if it's missing and in addition, the read method ensure the slash is present. I just changed the JavaScript URL guessing method to let the trailing slash.

          Oleg Nenashev added a comment -

          Fixed in Jenkins 2.127

          Oleg Nenashev added a comment - Fixed in Jenkins 2.127

          The way I understand this, this is to ensure the default is correct as soon as suggested by javascript so it does not have to be fixed on server side. This surely makes things better/easier though I doubt this is worth the backport presuming this does not correct any user-observable problem.

          Oliver Gondža added a comment - The way I understand this, this is to ensure the default is correct as soon as suggested by javascript so it does not have to be fixed on server side. This surely makes things better/easier though I doubt this is worth the backport presuming this does not correct any user-observable problem.

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

              Created:
              Updated:
              Resolved: