refs nextcloud#6962, refs nextcloud#5309; do not throw AppNotEnabledE…

…xception for app public pages Signed-off-by: Julien Veyssier <[email protected]>
julien-nc · Feb 28, 2018 · 352acd2 · 352acd2
1 parent 0cfdf47
commit 352acd2
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/private/AppFramework/Middleware/Security/SecurityMiddleware.php b/lib/private/AppFramework/Middleware/Security/SecurityMiddleware.php
@@ -195,8 +195,9 @@ public function beforeController($controller, $methodName) {
 		 * Checks if app is enabled (also includes a check whether user is allowed to access the resource)
 		 * The getAppPath() check is here since components such as settings also use the AppFramework and
 		 * therefore won't pass this check.
+		 * If page is public, app does not need to be enabled for current user/visitor
 		 */
-		if(\OC_App::getAppPath($this->appName) !== false && !$this->appManager->isEnabledForUser($this->appName)) {
+		if(\OC_App::getAppPath($this->appName) !== false && !$isPublicPage && !$this->appManager->isEnabledForUser($this->appName)) {
 			throw new AppNotEnabledException();
 		}