[JIRA] (JENKINS-59849) Spaces don't work in resource root paths

[jsoref+jenkins@gmail.com (JIRA)]

Josh Soref updated an issue

Jenkins / JENKINS-59849 Spaces don't work in resource root paths Change By: Josh Soref Summary: Parentheses Spaces don't work in resource root paths Add Comment

This message was sent by Atlassian Jira (v7.13.6#713006-sha1:cc4451f)

[dbeck@cloudbees.com (JIRA)]

Daniel Beck updated an issue

Jenkins / JENKINS-59849

Spaces don't work in resource root paths

Change By: Daniel Beck Issue Type: Improvement Bug

Add Comment

[dbeck@cloudbees.com (JIRA)]

Daniel Beck assigned an issue to Daniel Beck

Jenkins / JENKINS-59849

Spaces don't work in resource root paths

Change By: Daniel Beck Assignee: Daniel Beck

Add Comment

[dbeck@cloudbees.com (JIRA)]

Daniel Beck started work on JENKINS-59849

Change By: Daniel Beck Status: Open In Progress

Add Comment

[jsoref+jenkins@gmail.com (JIRA)]

Josh Soref updated an issue

Jenkins / JENKINS-59849

Spaces don't work in resource root paths

Change By: Josh Soref We have an archived folder called {{HeadlessChrome 77.0.3865 (Linux 0.0.0)}} with a file {{index.html}} the contents of that folder fail when served via resource root. If I make a copy of the folder {{HeadlessChrome-77.0.3865-(Linux-0.0.0)}}, that {{index.html}} file will be served successfully. Pipeline for testing: {code} node('master') {    sh '''mkdir "foo bar"    touch "foo bar/index.html"    '''    archiveArtifacts 'foo bar/' } {code}

Add Comment

[jsoref+jenkins@gmail.com (JIRA)]

Josh Soref updated an issue

Jenkins / JENKINS-59849 Spaces don't work in resource root paths Change By: Josh Soref

We have an archived folder called {{HeadlessChrome 77.0.3865 (Linux 0.0.0)}} with a file {{index.html}} the contents of that folder fail when served via resource root. If I make a copy of the folder {{HeadlessChrome-77.0.3865-(Linux-0.0.0)}}, that {{index.html}} file will be served successfully. Pipeline for testing:

{code :java } node('master') {    sh '''mkdir -p "foo bar"

touch "foo bar/index.html"    '''    archiveArtifacts 'foo bar/' } {code}

Add Comment

[jsoref+jenkins@gmail.com (JIRA)]

Josh Soref updated an issue

Jenkins / JENKINS-59849 Spaces don't work in resource root paths Change By: Josh Soref

We have an archived folder called {{HeadlessChrome 77.0.3865 (Linux 0.0.0)}} with a file {{index.html}} the contents of that folder fail when served via resource root. If I make a copy of the folder {{HeadlessChrome-77.0.3865-(Linux-0.0.0)}}, that {{index.html}} file will be served successfully. Pipeline for testing: {code:java} node('master') {

sh '''mkdir -p " a/ foo bar has lots of spaces to make people unhappy "    touch " a/ foo bar has lots of spaces to make people unhappy /index.html"    '''    archiveArtifacts ' foo bar a /' } {code}

Add Comment

[jsoref+jenkins@gmail.com (JIRA)]

Josh Soref commented on JENKINS-59849

Re: Spaces don't work in resource root paths Daniel Beck found the problem: https://github.com/jenkinsci/jenkins/blob/11c1146d0dc30e948ade8d1636ae01345c294291/core/src/main/java/jenkins/security/ResourceDomainRootAction.java#L166-L176

Add Comment

[dbeck@cloudbees.com (JIRA)]

Daniel Beck updated JENKINS-59849

Jenkins / JENKINS-59849

Spaces don't work in resource root paths

Change By: Daniel Beck Status: In Progress Review

Add Comment

[dbeck@cloudbees.com (JIRA)]

Daniel Beck updated JENKINS-59849

Will be released tomorrow or Monday.

Jenkins / JENKINS-59849 Spaces don't work in resource root paths

Change By: Daniel Beck Status: In Review Resolved Resolution: Fixed Released As: jenkins.2.201

Add Comment

[jsoref+jenkins@gmail.com (JIRA)]

Josh Soref closed an issue as Fixed

Picked up the change, verified it works. Thanks

Jenkins / JENKINS-59849 Spaces don't work in resource root paths

Change By: Josh Soref Status: Resolved Closed

Add Comment