allow setting user provided global credentials trough credentials popup
Signed-off-by: Robin Appelman <robin@icewind.nl>
This commit is contained in:
parent
e1ee36b4e8
commit
1a7ed9ae81
3 changed files with 15 additions and 4 deletions
|
@ -93,7 +93,8 @@ OCA.Files_External.StatusManager = {
|
|||
status: statusCode,
|
||||
id: mountData.id,
|
||||
error: statusMessage,
|
||||
userProvided: response.userProvided
|
||||
userProvided: response.userProvided,
|
||||
authMechanism: response.authMechanism
|
||||
};
|
||||
}
|
||||
afterCallback(mountData, self.mountStatus[mountData.mount_point]);
|
||||
|
@ -178,7 +179,7 @@ OCA.Files_External.StatusManager = {
|
|||
if (allMountStatus.hasOwnProperty(name) && allMountStatus[name].status > 0 && allMountStatus[name].status < 7) {
|
||||
var mountData = allMountStatus[name];
|
||||
if (mountData.type === "system") {
|
||||
if (mountData.userProvided) {
|
||||
if (mountData.userProvided || mountData.authMechanism === 'password::global::user') {
|
||||
// personal mount whit credentials problems
|
||||
this.showCredentialsDialog(name, mountData);
|
||||
} else {
|
||||
|
|
|
@ -27,6 +27,7 @@ namespace OCA\Files_External\Controller;
|
|||
|
||||
use OCA\Files_External\Lib\Auth\AuthMechanism;
|
||||
use OCA\Files_External\Lib\Auth\IUserProvided;
|
||||
use OCA\Files_External\Lib\Auth\Password\UserGlobalAuth;
|
||||
use OCA\Files_External\Lib\InsufficientDataForMeaningfulAnswerException;
|
||||
use OCP\ILogger;
|
||||
use \OCP\IRequest;
|
||||
|
@ -156,7 +157,7 @@ class UserGlobalStoragesController extends StoragesController {
|
|||
try {
|
||||
$storage = $this->service->getStorage($id);
|
||||
$authMechanism = $storage->getAuthMechanism();
|
||||
if ($authMechanism instanceof IUserProvided) {
|
||||
if ($authMechanism instanceof IUserProvided || $authMechanism instanceof UserGlobalAuth) {
|
||||
$authMechanism->saveBackendOptions($this->userSession->getUser(), $id, $backendOptions);
|
||||
$authMechanism->manipulateStorageConfig($storage, $this->userSession->getUser());
|
||||
} else {
|
||||
|
|
|
@ -44,11 +44,20 @@ class UserGlobalAuth extends AuthMechanism {
|
|||
|
||||
$this
|
||||
->setIdentifier('password::global::user')
|
||||
->setVisibility(BackendService::VISIBILITY_ADMIN)
|
||||
->setVisibility(BackendService::VISIBILITY_DEFAULT)
|
||||
->setScheme(self::SCHEME_PASSWORD)
|
||||
->setText($l->t('Global credentials, user entered'));
|
||||
}
|
||||
|
||||
public function saveBackendOptions(IUser $user, $id, $backendOptions) {
|
||||
// make sure we're not setting any unexpected keys
|
||||
$credentials = [
|
||||
'user' => $backendOptions['user'],
|
||||
'password' => $backendOptions['password'],
|
||||
];
|
||||
$this->credentialsManager->store($user->getUID(), self::CREDENTIALS_IDENTIFIER, $credentials);
|
||||
}
|
||||
|
||||
public function manipulateStorageConfig(StorageConfig &$storage, IUser $user = null) {
|
||||
if ($user === null) {
|
||||
throw new InsufficientDataForMeaningfulAnswerException('No credentials saved');
|
||||
|
|
Loading…
Reference in a new issue