use Net::IP;
use MIME::Base64;
use Digest::SHA;
-use Digest::HMAC_SHA1;
use URI::Escape;
use LWP::UserAgent;
use PVE::Tools qw(run_command lock_file file_get_contents split_list safe_print);
my $randomstr = "PVESPICE:$timestamp:$vmid:$node:" . rand(10);
- # this should be uses as one-time password
+ # this should be used as one-time password
# max length is 60 chars (spice limit)
# we pass this to qemu set_pasword and limit lifetime there
# keep this secret
my $ticket = Digest::SHA::sha1_hex($rsa_priv->sign($randomstr));
# Note: spice proxy connects with HTTP, so $proxyticket is exposed to public
- # we use a signature/timestamp to make sure nobody can fake such ticket
+ # we use a signature/timestamp to make sure nobody can fake such a ticket
# an attacker can use this $proxyticket, but he will fail because $ticket is
# private.
- # The proxy need to be able to extract/verify the ticket
+ # The proxy needs to be able to extract/verify the ticket
# Note: data needs to be lower case only, because virt-viewer needs that
- # Note: RSA signature are too long (>=256 charaters) and makes problems with remote-viewer
+ # Note: RSA signature are too long (>=256 charaters) and make problems with remote-viewer
my $secret = &$get_csrfr_secret();
my $plain = "pvespiceproxy:$timestamp:$vmid:" . lc($node);
# read x509 subject
my $bio = Net::SSLeay::BIO_new_file($filename, 'r');
+ die "Could not open $filename using OpenSSL\n"
+ if !$bio;
+
my $x509 = Net::SSLeay::PEM_read_bio_X509($bio);
Net::SSLeay::BIO_free($bio);
+
+ die "Could not parse X509 certificate in $filename\n"
+ if !$x509;
+
my $nameobj = Net::SSLeay::X509_get_subject_name($x509);
my $subject = Net::SSLeay::X509_NAME_oneline($nameobj);
Net::SSLeay::X509_free($x509);
'release-cursor' => "Ctrl+Alt+R",
type => 'spice',
title => $title,
- host => $proxyticket, # this break tls hostname verification, so we need to use 'host-subject'
+ host => $proxyticket, # this breaks tls hostname verification, so we need to use 'host-subject'
proxy => "http://$proxy:3128",
'tls-port' => $port,
'host-subject' => $subject,
my $type = $tfa_cfg->{type};
- die "missing one time password for Factor-two authentication '$type'\n" if !$otp;
+ die "missing one time password for two-factor authentication '$type'\n" if !$otp;
# fixme: proxy support?
my $proxy;
}
# password should be utf8 encoded
-# Note: some pluging delay/sleep if auth fails
+# Note: some plugins delay/sleep if auth fails
sub authenticate_user {
my ($username, $password, $otp) = @_;
my $domain_cfg = cfs_read_file('domains.cfg');
my $cfg = $domain_cfg->{ids}->{$realm};
- die "auth domain '$realm' does not exists\n" if !$cfg;
+ die "auth domain '$realm' does not exist\n" if !$cfg;
my $plugin = PVE::Auth::Plugin->lookup($cfg->{type});
$plugin->store_password($cfg, $realm, $username, $password);
}
# into 3 groups (per category)
# root: only root is allowed to do that
# admin: an administrator can to that
-# user: a normak user/customer can to that
+# user: a normal user/customer can to that
my $privgroups = {
VM => {
root => [],
my $valid_privs = {};
my $special_roles = {
- 'NoAccess' => {}, # no priviledges
- 'Administrator' => $valid_privs, # all priviledges
+ 'NoAccess' => {}, # no privileges
+ 'Administrator' => $valid_privs, # all privileges
};
sub create_roles {
if (defined ($valid_privs->{$priv})) {
$usercfg->{roles}->{$role}->{$priv} = 1;
} else {
- die "invalid priviledge '$priv'\n";
+ die "invalid privilege '$priv'\n";
}
}
}
my ($priv, $noerr) = @_;
if (!$valid_privs->{$priv}) {
- die "invalid priviledge '$priv'\n" if !$noerr;
+ die "invalid privilege '$priv'\n" if !$noerr;
return undef;
}
}
foreach my $user (keys %{$d->{users}}) {
- # no need to save, because root is always 'Administartor'
+ # no need to save, because root is always 'Administrator'
next if $user eq 'root@pam';
my $l0 = '';
$paramstr .= "$key=$param->{$key}";
}
- my $sig = uri_escape(encode_base64(Digest::HMAC_SHA1::hmac_sha1($paramstr, decode_base64($api_key || '')), ''));
+ # hmac_sha1_base64 does not add '=' padding characters, so we use encode_base64
+ my $sig = uri_escape(encode_base64(Digest::SHA::hmac_sha1($paramstr, decode_base64($api_key || '')), ''));
return ($paramstr, $sig);
}
die "yubico: missing API KEY\n" if !defined($api_key);
die "yubico: no associated yubico keys\n" if $keys =~ m/^\s+$/;
- die "yubico: wrong OTP lenght\n" if (length($otp) < 32) || (length($otp) > 48);
-
- # we always use http, because https cert verification always make problem, and
- # some proxies does not work with https.
+ die "yubico: wrong OTP length\n" if (length($otp) < 32) || (length($otp) > 48);
$url = 'http://api2.yubico.com/wsapi/2.0/verify' if !defined($url);
my $params = {
- nonce => Digest::HMAC_SHA1::hmac_sha1_hex(time(), rand()),
+ nonce => Digest::SHA::hmac_sha1_hex(time(), rand()),
id => $api_id,
otp => uri_escape($otp),
timestamp => 1,
my $req = HTTP::Request->new('GET' => "$url?$paramstr");
- my $ua = LWP::UserAgent->new(protocols_allowed => ['http'], timeout => 30);
+ my $ua = LWP::UserAgent->new(protocols_allowed => ['http', 'https'], timeout => 30);
if ($proxy) {
- $ua->proxy(['http'], $proxy);
+ $ua->proxy(['http', 'https'], $proxy);
} else {
$ua->env_proxy;
}