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

Encryption2 #50

Closed
wants to merge 3 commits into from
Closed

Encryption2 #50

wants to merge 3 commits into from

Conversation

lzap
Copy link
Contributor

@lzap lzap commented Apr 27, 2012

So I am adding support for plain text passwords encryption (please note this is encryption for passwords we need to keep - obscurity not security).

  1. There is a new utility called katello-passwd (see man page or help option) to encrypt a password.

  2. It starts with $1$ (1 is for version 1) and it can only be used for database-password setting (we do not have any plaintext passwords in Katello yet)

  3. Katello can also read unencrypted passwords from this field (if it does not start with $1$)

  4. I will raise a RFE against Candlepin to do the same for database password. Pulp is fine (no passwords there).

  5. I modified katello-configure to use the function to encrypt database password during installation

More on the demo.

@lzap
Copy link
Contributor Author

lzap commented Apr 27, 2012

Raised RFE for Candlepin: https://bugzilla.redhat.com/show_bug.cgi?id=816935
Pulp has no plaintext passwords currently.

@lzap
Copy link
Contributor Author

lzap commented Apr 27, 2012

In future we might want to store passphrase in a file setting a strict SELinux policy to it, but since multiple processes (e.g. unconfined tomcat) from different policies would need to access it, I'd recommend to stick with this approach.

@iNecas
Copy link
Member

iNecas commented Apr 30, 2012

What if the pain password begins with $1$? Not very probable but one never knows.

@lzap
Copy link
Contributor Author

lzap commented Apr 30, 2012

@iNecas If plain password begins with $1$ it will work too. ;-)

@jsomara
Copy link
Contributor

jsomara commented Apr 30, 2012

This pull request needs to be rebased (This pull request cannot be automatically merged.)

@lzap
Copy link
Contributor Author

lzap commented May 2, 2012

Before I will rebase it again (third time), let's give me GOs!

@iNecas
Copy link
Member

iNecas commented May 2, 2012

It's working indeed. GO from me. Lets do the final rebase.

@lzap
Copy link
Contributor Author

lzap commented May 10, 2012

Closing for now, I want to rebase and also add passphase file.

@lzap lzap closed this May 10, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants