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.

Sign up for GitHub

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

Jump to bottom

Scramble password #6

Merged

ohtake merged 2 commits into jenkinsci:master from ohtake:scramble-password

Jun 11, 2012

Member

ohtake commented Jun 8, 2012

Deploy plugin saves plain passwords into config.xml.
Better to use Scrambler.

These changes are backward-compatible but are not forward-compatible.

buildhive commented Jun 8, 2012

Jenkins » deploy-plugin #2 SUCCESS
This pull request looks good
(what's this?)

Member Author

ohtake commented Jun 10, 2012

ohtake@dd7f823#commitcomment-1437724

this should be x.initCause(e). not x

Fixed and rebased. Thank you.

buildhive commented Jun 10, 2012

Jenkins » deploy-plugin #3 SUCCESS
This pull request looks good
(what's this?)

ohtake added 2 commits

June 10, 2012 22:11


          Encapsulate password field

95ddcb6


          Scramble password

400602b

buildhive commented Jun 10, 2012

Jenkins » deploy-plugin #4 SUCCESS
This pull request looks good
(what's this?)

ohtake merged commit 400602b into jenkinsci:master

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

Labels

None yet