throw exception if private key is missing
This commit is contained in:
parent
0580c232d7
commit
c2a45c1238
4 changed files with 21 additions and 1 deletions
|
@ -13,6 +13,7 @@ OC::$CLASSPATH['OCA\Encryption\Helper'] = 'files_encryption/lib/helper.php';
|
|||
// Exceptions
|
||||
OC::$CLASSPATH['OCA\Encryption\Exceptions\MultiKeyEncryptException'] = 'files_encryption/lib/exceptions.php';
|
||||
OC::$CLASSPATH['OCA\Encryption\Exceptions\MultiKeyDecryptException'] = 'files_encryption/lib/exceptions.php';
|
||||
OC::$CLASSPATH['OCA\Encryption\Exceptions\EncryptionException'] = 'files_encryption/lib/exceptions.php';
|
||||
|
||||
\OCP\Util::addTranslations('files_encryption');
|
||||
\OCP\Util::addscript('files_encryption', 'encryption');
|
||||
|
|
|
@ -30,8 +30,16 @@ namespace OCA\Encryption\Exceptions;
|
|||
* 30 - encryption header to large
|
||||
* 40 - unknown cipher
|
||||
* 50 - encryption failed
|
||||
* 60 - no private key available
|
||||
*/
|
||||
class EncryptionException extends \Exception {
|
||||
const UNEXPECTED_END_OF_ENCRTYPTION_HEADER = 10;
|
||||
const UNEXPECTED_BLOG_SIZE = 20;
|
||||
const ENCRYPTION_HEADER_TO_LARGE = 30;
|
||||
const UNKNOWN_CIPHER = 40;
|
||||
const ENCRYPTION_FAILED = 50;
|
||||
const NO_PRIVATE_KEY_AVAILABLE = 60;
|
||||
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
@ -30,6 +30,7 @@
|
|||
*/
|
||||
|
||||
namespace OCA\Encryption;
|
||||
use OCA\Encryption\Exceptions\EncryptionException;
|
||||
|
||||
/**
|
||||
* Provides 'crypt://' stream wrapper protocol.
|
||||
|
@ -106,6 +107,10 @@ class Stream {
|
|||
$this->session = new \OCA\Encryption\Session($this->rootView);
|
||||
|
||||
$this->privateKey = $this->session->getPrivateKey();
|
||||
if ($this->privateKey === false) {
|
||||
throw new EncryptionException('Session does not contain a private key, maybe your login password changed?',
|
||||
EncryptionException::NO_PRIVATE_KEY_AVAILABLE);
|
||||
}
|
||||
|
||||
$normalizedPath = \OC\Files\Filesystem::normalizePath(str_replace('crypt://', '', $path));
|
||||
if ($originalFile = Helper::getPathFromTmpFile($normalizedPath)) {
|
||||
|
|
|
@ -100,6 +100,8 @@ class OC_Connector_Sabre_File extends OC_Connector_Sabre_Node implements \Sabre\
|
|||
} catch (\OCP\Files\LockNotAcquiredException $e) {
|
||||
// the file is currently being written to by another process
|
||||
throw new OC_Connector_Sabre_Exception_FileLocked($e->getMessage(), $e->getCode(), $e);
|
||||
} catch (\OCA\Encryption\Exceptions\EncryptionException $e) {
|
||||
throw new \Sabre\DAV\Exception\Forbidden($e->getMessage());
|
||||
}
|
||||
|
||||
// if content length is sent by client:
|
||||
|
@ -152,7 +154,11 @@ class OC_Connector_Sabre_File extends OC_Connector_Sabre_Node implements \Sabre\
|
|||
if (\OC_Util::encryptedFiles()) {
|
||||
throw new \Sabre\DAV\Exception\ServiceUnavailable();
|
||||
} else {
|
||||
return $this->fileView->fopen(ltrim($this->path, '/'), 'rb');
|
||||
try {
|
||||
return $this->fileView->fopen(ltrim($this->path, '/'), 'rb');
|
||||
} catch (\OCA\Encryption\Exceptions\EncryptionException $e) {
|
||||
throw new \Sabre\DAV\Exception\Forbidden($e->getMessage());
|
||||
}
|
||||
}
|
||||
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue