Solved

Multi Factor Authentication (MFA) Flaw


Badge +1

I started looking at the MFA on Command Centre and baffled as it is flawed. If my domain account has been compromised, I would be expecting the second factor to be the 2nd line of defence. But no, you can request a new pin that gets sent to your compromised domain account e-mail address. I then looked to see if I can amend my account by adding an external e-mail address, but LDAP pulls this from the domain and can not be edited. By editing the e-mail script we can omit the pin, but I think this hasn’t been thought through by Commvault, considering that backups are supposed to be the last line of defence against a cyber attack the two factor serves only to delay the time it takes for SMTP to deliver a new pin.

icon

Best answer by Anand 7 June 2021, 21:32

@dude  In other words, yubikey option doesn’t get invoke, when we do SSO.  In case of ADFS, yubikey support need to be added in AD. When token comes back to us, user is already marked authenticated. Internally we have tried yubikey with Azure AD setup and it worked fine. Currently we are adding feature to allow local or direct AD user authentication with yubikey.

Editing in FR25 to mark this as the best (and complete) answer.

 

 

 

 

View original

19 replies

Userlevel 4
Badge +8

This is an issue when using domain user accounts. - If you omit the PIN from the Email, you can use one of the supported App’s to generate the PIN.

For Domain accounts you could give a “View Only” role to avoid any compromised accounts making changes.


You could use a local user account (non-domain) that has privileges in the CommCell and add the users email address to that local account. 

Then to be compromised they would have to obtain the Commvault local credentials and the user’s domain credentials for the PIN email.

 

Best Regards,

Michael

Userlevel 7
Badge +15

@Dancer , I’ll speak to some internal folks about this.

Badge +1

That was meant to say “considering backups is supposed to be the last line of defence”

Userlevel 7
Badge +15

That was meant to say “considering backups is supposed to be the last line of defence”

I edited the original post for you :grin:

I have asked some devs to take a look at this thread as well!

Badge

we are checking this internally with

 @Divya Trivedi@Tharun Pathepuram  @Anand 


we will get back on this.

Userlevel 1
Badge

I started looking at the MFA on Command Centre and baffled as it is flawed. If my domain account has been compromised, I would be expecting the second factor to be the 2nd line of defence. But no, you can request a new pin that gets sent to your compromised domain account e-mail address. I then looked to see if I can amend my account by adding an external e-mail address, but LDAP pulls this from the domain and can not be edited. By editing the e-mail script we can omit the pin, but I think this hasn’t been thought through by Commvault, considering that backups are supposed to be the last line of defence against a cyber attack the two factor serves only to delay the time it takes for SMTP to deliver a new pin.

This is being worked on. 

  1. Option to block the pin to be sent in email.
  2. Option to set the email for the AD/LDAP account if user has rights.
Badge +1

Great stuff. Maybe the AD/LDAP email address should stay for the purpose of scheduled reports etc, but a 2nd e-mail field used exclusively for recovery purposes. Users should be flagged that their recovery email address is empty if MFA is enabled, and periodically prompted (6 months) to check their details are still correct. I also think any manual amending of the e-mail address be limited to the user, therefore a administrator can create\disable an account but not amend it.

Badge +1

We also just discovered disabling SSO has forced the java client to prompt for 2 factor auth. For wider community as I presume you know this.

Userlevel 1
Badge

Great stuff. Maybe the AD/LDAP email address should stay for the purpose of scheduled reports etc, but a 2nd e-mail field used exclusively for recovery purposes. Users should be flagged that their recovery email address is empty if MFA is enabled, and periodically prompted (6 months) to check their details are still correct. I also think any manual amending of the e-mail address be limited to the user, therefore a administrator can create\disable an account but not amend it.

I understand that you want end user to amend the email but admin may not like it. They would like to keep the user email same as AD email to avoid defragmentation, data leak or security issues. We are targeting to update the email only when it’s empty. Ex- service account, SAML account etc. 

Userlevel 1
Badge

We also just discovered disabling SSO has forced the java client to prompt for 2 factor auth. For wider community as I presume you know this.

Second factor is asked only when CV does the password authentication. When password is validated thru SSO or IDP, we act as relying party. 

@Anand any way to get ADFS and allow Yubikeys?

@Mike Struening just wondering if you would be able to look my question above about ADFS/Yubikey and check internally? Thanks

Userlevel 1
Badge

@dude  In other words, yubikey option doesn’t get invoke, when we do SSO.  In case of ADFS, yubikey support need to be added in AD. When token comes back to us, user is already marked authenticated. Internally we have tried yubikey with Azure AD setup and it worked fine. Currently we are adding feature to allow local or direct AD user authentication with yubikey.

Editing in FR25 to mark this as the best (and complete) answer.

 

 

 

 

Userlevel 7
Badge +15

@dude , let me know if @Anand ‘s reply satisfied your question :nerd:

@dude  In other words, yubikey option doesn’t get invoke, when we do SSO.  In case of ADFS, yubikey support need to be added in AD. When token comes back to us, user is already marked authenticated. Internally we have tried yubikey with Azure AD setup and it worked fine. Currently we are adding feature to allow local or direct AD user authentication with yubikey.

 

 

 

 

@Anand I`m not sure I understand. It is currently not supported but it is coming in future release? do you  know when?

Userlevel 7
Badge +15

Hey @Anand , do you have any ETA on release for @dude ?

Userlevel 7
Badge +15

@dude , this is coming inFR25.

@dude , this is coming inFR25.

Thanks

Userlevel 7
Badge +15

Anytime, man!

Reply