Merge pull request #861 from nextcloud/backport/860/stable33

[stable33] refactor: simplify enabled-state check in validateExAppRequestToNC

Author: oleksandr-nc

lib/Service/AppAPIService.php

@@ -302,17 +302,15 @@ public function validateExAppRequestToNC(IRequest $request, bool $isDav = false)
 		$authValid = $authorizationSecret === $exApp->getSecret();
 
 		if ($authValid) {
-			if (!$isDav) {
-				try {
-					$path = $request->getPathInfo();
-				} catch (\Exception $e) {
-					$this->logger->error(sprintf('Error getting path info. Error: %s', $e->getMessage()), ['exception' => $e]);
-					return false;
-				}
-				if (!$exApp->getEnabled() && !$this->isExemptFromEnabledCheck($path, $exApp)) {
-					$this->logger->error(sprintf('ExApp with appId %s is disabled (%s)', $request->getHeader('EX-APP-ID'), $request->getRequestUri()));
-					return false;
-				}
+			try {
+				$path = $request->getPathInfo();
+			} catch (\Exception $e) {
+				$this->logger->error(sprintf('Error getting path info. Error: %s', $e->getMessage()), ['exception' => $e]);
+				return false;
+			}
+			if (!$exApp->getEnabled() && !$this->isExemptFromEnabledCheck($path, $exApp)) {
+				$this->logger->error(sprintf('ExApp with appId %s is disabled (%s)', $request->getHeader('EX-APP-ID'), $request->getRequestUri()));
+				return false;
 			}
 			if (!$this->handleExAppVersionChange($request, $exApp)) {
 				return false;