Truncate 2F device name (#1386)

This commit is contained in:
Christophe Maudoux 2018-04-05 19:43:06 +02:00
parent e2effb46a1
commit 5bd5920b0f
3 changed files with 13 additions and 4 deletions

View File

@ -67,8 +67,11 @@ sub run {
my $TOTPName = $req->param('TOTPName');
my $epoch = time();
# Set default name if empty
# Set default name if empty and troncate name if too long
$TOTPName ||= $epoch;
$TOTPName =
substr( $TOTPName, 0, $self->conf->{max2FDevicesNameLength} );
$self->logger->debug("TOTP name : $TOTPName");
unless ($code) {
$self->logger->userInfo('TOTP registration: empty validation form');

View File

@ -104,8 +104,10 @@ sub run {
my $keyName = $req->param('keyName');
my $epoch = time();
# Set default name if empty
# Set default name if empty and troncate name if too long
$keyName ||= $epoch;
$keyName =
substr( $keyName, 0, $self->conf->{max2FDevicesNameLength} );
$self->logger->debug("Key name : $keyName");
push @{$_2FDevices},

View File

@ -23,7 +23,8 @@ has logo => ( is => 'rw', default => 'yubikey.png' );
sub init {
my ($self) = @_;
$self->conf->{yubikey2fPublicIDSize} ||= 12;
#$self->conf->{yubikey2fPublicIDSize} ||= 12;
return 1;
}
@ -37,8 +38,11 @@ sub run {
my $UBKName = $req->param('UBKName');
my $epoch = time();
# Set default name if empty
# Set default name if empty and troncate name if too long
$UBKName ||= $epoch;
$UBKName = substr( $UBKName, 0, $self->conf->{max2FDevicesNameLength} );
$self->logger->debug("Yubikey name : $UBKName");
if ( $otp
and length($otp) > $self->conf->{yubikey2fPublicIDSize} )
{