Security Update, make --user/--password optional - Githubissues

rdrgmnzs / pulp_centos_errata_import

Imports CentOS (from http://cefs.steve-meier.de/) errata into Pulp / Katello

61 stars 22 forks source link

Security Update, make --user/--password optional #22

Closed rdrgmnzs closed 7 years ago

rdrgmnzs commented 7 years ago

Passing passwords on the command line is a security risk. Make --user/--password optional in favor of using ~/.pulp/client-cert.pem or ~/.pulp/admin.conf.

errata_import.pl:

Make --user/--password optional and update usage
Add a warning about using --user/--password
Add $pulp_args and populate if --user/--password are used
If user and password are NOT supplied verify one of ~/.pulp/user-cert.pem or ~/.pulp/admin.conf are available

sample_wrapper_script.sh:

show usage without user/password
show usage with pulp-admin repo list | aws | xargs

README.md:

Update examples
Update for making --user/--password optional
Add section Authentication options

rdrgmnzs commented 7 years ago

@alanwevans thanks for all the work you put into this.