Skip to content

download source tap file from build dir#5

Open
forrest79 wants to merge 2 commits into
jenkinsci:masterfrom
forrest79:download-source-from-build-dir
Open

download source tap file from build dir#5
forrest79 wants to merge 2 commits into
jenkinsci:masterfrom
forrest79:download-source-from-build-dir

Conversation

@forrest79

Copy link
Copy Markdown

This modified TAP Extended Test Results, source tap file, that can be viewed from this page is loaded from workspace, where is always newest file. According to me, what user want, is file from build, where user is. This modification download source tap file from build directory.

Maybe it could be done in simpliest way :-) Unfortunately I am not an expert on Jenkins plugins :-)

@cloudbees-pull-request-builder

Copy link
Copy Markdown

plugins » tap-plugin #55 SUCCESS
This pull request looks good

@jenkinsadmin

Copy link
Copy Markdown
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@kinow

kinow commented Mar 5, 2014

Copy link
Copy Markdown
Member

Hi @forrest79!

We had this behaviour in the past and I removed it in favour of the /view/ URL's because it was possible to access other files via the contents URL (my fault for adding this security bug, of course).

I'll give a shot to your pull request in the next days and will merge it after that. I think the build.getRootDir and FilePath together will do the trick.

Thanks!

@avdv

avdv commented Jun 16, 2015

Copy link
Copy Markdown

👍 I want this too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants