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

Pulp::Admin::Login/Exec[pulp-auth] doesn't seem to handle expired session? #323

Open
TJM opened this issue Jul 2, 2018 · 2 comments
Open

Comments

@TJM
Copy link
Member

TJM commented Jul 2, 2018

Apparently when pulp::admin::login runs, it gets a "session" certificate for some time limited period, but doesn't check whether it has expired or not, causing further steps to get an exception as follows:

Error: Failed to apply catalog: https request threw exception SSL_connect returned=1 errno=0 state=unknown state: sslv3 alert certificate expired. Connection details: url=https://pulp.domain.corp/pulp/api/v2/repositories/, cert_path=/root/.pulp/user-cert.pem.
@sentinel159868
Copy link

Same issue here..... any workarounds to handle it?

@TJM
Copy link
Member Author

TJM commented Sep 13, 2018

As a workaround, you could remove the file (/root/.pulp/user-cert.pem), which will cause the login process to run again, that will work until the session times out again. :)

It might be worth augmenting the login class to create a /root/.pulp/admin.conf with the credentials?

Looks like its already there, just have to set pulp::admin::login_method: file (strike that, it looks like login_method is set in code.

if $enable_admin {
if $ssl_username and $ssl_username != '' {
warning('Using $ssl_username means pulp-admin login doesn\'t work. Falling back to file login but pulp_*repo providers won\'t work')
$login_method = 'file'
} else {
$login_method = 'login'
}
class { '::pulp::admin':
enable_deb => $enable_deb,
enable_docker => $enable_docker,
enable_nodes => $enable_parent_node,
enable_ostree => $enable_ostree,
enable_puppet => $enable_puppet,
enable_python => $enable_python,
enable_rpm => $enable_rpm,
ca_path => $ca_cert,
login_method => $login_method,
username => $default_login,
password => $default_password,
require => Class['pulp::apache'],
}

I suppose you could set pulp::enable_admin: false and then include pulp::admin with login_method set to files, but it claims there that setting will break the providers :(

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

No branches or pull requests

2 participants