Fix readfile with exit

Author: rookie0

appinfo/info.xml

@@ -7,7 +7,7 @@
     <description>
         <![CDATA[Files sharing use file stored path. Now you cau use nextcloud as your CDN origin.
         What you need to do is just share your files, then click Open Sharing Path to access from files dropdown actions menu. ]]></description>
-    <version>0.2.0</version>
+    <version>0.2.1</version>
     <licence>agpl</licence>
     <author mail="[email protected]">Rookie0</author>
     <namespace>SharingPath</namespace>
@@ -19,7 +19,7 @@
         https://user-images.githubusercontent.com/5813232/61992484-bc745d80-b091-11e9-84bc-005a2a6caf14.png
     </screenshot>
     <dependencies>
-        <nextcloud min-version="15" max-version="18"/>
+        <nextcloud min-version="15" max-version="19"/>
     </dependencies>
 
     <settings>

lib/Controller/PathController.php

@@ -135,14 +135,19 @@ public function handle($uid, $path)
                 }
             }
 
+            // FIXME: The exit is required here because otherwise the AppFramework is trying to add headers as well
             $view->readfile($path);
+            exit;
         } catch (NotFoundException $e) {
             http_response_code(404);
-            $this->logger->warning("request user {$uid} file {$path} not found.");
+            $this->logger->warning("request user {$uid} file {$path} not found.", ['app' => Application::APP_ID]);
             exit;
         } catch (\Exception $e) {
             http_response_code(500);
-            $this->logger->error("request user {$uid} file {$path} failed: " . $e->getMessage(), ['extra_context' => $e->getTrace()]);
+            $this->logger->error("request user {$uid} file {$path} failed: " . $e->getMessage(), [
+                'app'           => Application::APP_ID,
+                'extra_context' => $e->getTrace(),
+            ]);
             exit;
         }
     }