[pve-devel] applied: [PATCH manager 1/2] fix wrongly disabled TFA Window

Thomas Lamprecht t.lamprecht at proxmox.com
Wed Apr 17 15:02:15 CEST 2019


On 4/17/19 2:28 PM, Dominik Csapak wrote:
> if we have no info about TFA in the userview (x as key instead of
> x!oath or x!u2f) we disabled the whole window and the only action
> was to delete
> 
> instead show all options, so the user can overwrite the setting
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  www/manager6/dc/TFAEdit.js | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/www/manager6/dc/TFAEdit.js b/www/manager6/dc/TFAEdit.js
> index 6d58658c..c57c5339 100644
> --- a/www/manager6/dc/TFAEdit.js
> +++ b/www/manager6/dc/TFAEdit.js
> @@ -110,11 +110,11 @@ Ext.define('PVE.window.TFAEdit', {
>  	    },
>  	    canSetupTOTP: function(get) {
>  		var tfa = get('tfa_type');
> -		return (tfa === undefined || tfa === 'totp');
> +		return (tfa === undefined || tfa === 'totp' || tfa === 1);
>  	    },
>  	    canSetupU2F: function(get) {
>  		var tfa = get('tfa_type');
> -		return (get('u2f_available') && (tfa === undefined || tfa === 'u2f'));
> +		return (get('u2f_available') && (tfa === undefined || tfa === 'u2f' || tfa === 1));
>  	    }
>  	}
>      },
> 

applied, thanks!




More information about the pve-devel mailing list