[TH] Install and Upgrade [Deleted]

If the password is actually being printed back out, it would be in the page source; inspecting it may not show as a browser design choice. I don't know the implementation to know whether or not that's the case though.
 
If the password is actually being printed back out, it would be in the page source; inspecting it may not show as a browser design choice. I don't know the implementation to know whether or not that's the case though.
Just checked (look at me being all helpful :p) Starred out in Page Source too.
 
I was the one who noticed it. I can still reproduce this in safari, by clicking on the "stored credentials" option in ACP, clicking edit on the desired credential and then right clicking in the password field and inspecting. In the screenshot, you can see username is test and password is "hidemetest". Not major if you are the only admin with ACP access but unsafe if you aren't.
 

Attachments

  • Screen Shot 2015-05-30 at 8.27.33 PM.webp
    Screen Shot 2015-05-30 at 8.27.33 PM.webp
    78.1 KB · Views: 22
I was the one who noticed it. I can still reproduce this in safari, by clicking on the "stored credentials" option in ACP, clicking edit on the desired credential and then right clicking in the password field and inspecting. In the screenshot, you can see username is test and password is "hidemetest". Not major if you are the only admin with ACP access but unsafe if you aren't.
Okay, I didn't have the whole edit the field first part of the equation :)

Yes, I can confirm this.
 
Anyone with the premium version that can test as well? I know it says you can enable encryption of the password in the database with the premium version but wondering if it still shows in the ACP using this method.

Okay, I didn't have the whole edit the field first part of the equation :)

Yes, I can confirm this.
Yeah, sorry. I wasn't as clear about the procedure when @Brogan and myself were talking about it.
 
How privacy is provided account? Do you store this information on your server (hostname,username,pass etc)
multi admin site to privacy issue Ftp Password text password visible
Solution store credential function?
 
Last edited:
@Jon W Hey, how does the copyright system work actually? I've made sure to allow copyright to appear in all cases and never hide it, but for the past few days, I've never seen the copyright in my footer at all. It behaves this way both on my live forums and my local host.
 
@TDUBS However you can prevent the Stored Credentials page to return the actual password when someone tries to edit it by editing this file:
library\Waindigo\InstallUpgrade\Extend\XenForo\ControllerAdmin\AddOn.php
The function that returns the password begins on line 497 (function name: actionStoredCredentialsEdit). Preventing the function from returning the actual password should be enough if all you worry about is other admins that have access to Admin CP from accessing the password.

On my case, I added this line:
$credential['password'] = "fake password";
right before this line (adding it after will also work - just make sure it's changed before being added to $viewParams array):
$credential = $installUpgradeModel->prepareCredential($credential);
 
Last edited:
@TDUBS However you can prevent the Stored Credentials page to return the actual password when someone tries to edit it by editing this file:

The function that returns the password begins on line 497. Preventing the function from returning the actual password should be enough if all you worry about is other admins that have access to Admin CP from accessing the password.

So this is the fix Jon would have to do in order to fix this problem?
 
@TDUBS It will only solve the issue that those users above reported. The issue where it's possible for other admins that have access to Admin CP to retrieve password of the stored credentials (and if you're not on an encrypted connection, it's also possible for MITM attacker to retrieve it though).
 
Last edited:
@TDUBS It will only solve the issue that those users above reported. The issue where it's possible for other admins that have access to Admin CP to retrieve password of the stored credentials (and if you're not on an encrypted connection, it's also possible for MIM attacker to retrieve it though).

I'll hold off on this then. Security through obscurity.
 
Last edited:
Also, it would be better if the add-on didn't return the actual FTP password on FTP Login Details option. And also the XenForo License Validation Token or even the XenForo License Key. For example, Audentio's UI.X has a better method. It won't return Audentio API key once it has been set. It only allows the user to change it, not view it. I'm not so concerned since I especially set the security setting of my admin.php on CloudFlare to High. And also the encryption of my whole forums is fully strict. But still, there's a concern since I'm not the only one who have access to Admin CP, and I can't tell for sure that the other admins' devices are safe.
 
Top Bottom