Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[FIXED JENKINS-19752] write in the correct outputstream #954

Closed
wants to merge 2 commits into from
Closed

[FIXED JENKINS-19752] write in the correct outputstream #954

wants to merge 2 commits into from

Conversation

mathieu-pousse
Copy link

This fixes the isse that appears in 1.532 and that corrupts all the zip generated by DirectoryBrowserSupport (https://issues.jenkins-ci.org/browse/JENKINS-19752)

http://job/zenika/lastSuccessfulBuild/artifact/repository/output/_zip_/output.zip

The process was not writing in the correct output stream.
This issue is quite annoying as you cannot download anymore the artifacts in a single file (zip)

@cloudbees-pull-request-builder

core » jenkins_main_trunk #1231 UNSTABLE
Looks like there's a problem with this pull request

@mathieu-pousse
Copy link
Author

it looks like this test is randomly failing... The pull request is not related to the User management or anything like that...

And the coverage report of this test shows us that the modified class DirectoryBrowserSupport is not used...


I fixed the unit test to wait until the job is complete, otherwise it is filtered and user does not belong to the build. It was clearly depending on the server load.

@cloudbees-pull-request-builder

core » jenkins_main_trunk #1233 UNSTABLE
Looks like there's a problem with this pull request

olivergondza added a commit that referenced this pull request Sep 26, 2013
@olivergondza
Copy link
Member

Merged commit 11fe787 fixing JENKINS-19752. The test failure was, as you said, unrelated to your changes. I will create a proper fix later. Thanks for contributing.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants