Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix the CLI test failure #3406

Merged

Conversation

@Wadeck
Copy link
Contributor

Wadeck commented Apr 26, 2018

Small fix to avoid a test to fail. Initially due to an invisible merge conflict (new feature + separate correction, the correction was not aware of the new feature).

@reviewbybees

@daniel-beck
Copy link
Member

daniel-beck commented Apr 26, 2018

Seems to be a bad backport in 8a2f986, line 271 needed to be adapted. Originally introduced for a security fix.

@olivergondza olivergondza merged commit 3485779 into jenkinsci:stable-2.107 Apr 26, 2018
1 check was pending
1 check was pending
continuous-integration/jenkins/pr-merge This commit is being built
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.