[stable8.1] Set content type when downloading log file to force downl…

…oad in some browsers (#25382)
owncloud · Jul 6, 2016 · 044ee07 · 044ee07
1 parent 7b92dd2
commit 044ee07
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/settings/controller/logsettingscontroller.php b/settings/controller/logsettingscontroller.php
@@ -103,6 +103,7 @@ public function getEntries($count=50, $offset=0) {
 	 */
 	public function download() {
 		$resp = new StreamResponse(\OC_Log_Owncloud::getLogFilePath());
+		$resp->addHeader('Content-Type', 'application/octet-stream');
 		$resp->addHeader('Content-Disposition', 'attachment; filename="owncloud.log"');
 		return $resp;
 	}

diff --git a/tests/settings/controller/logsettingscontrollertest.php b/tests/settings/controller/logsettingscontrollertest.php
@@ -70,5 +70,8 @@ public function testDownload() {
 		$response = $this->logSettingsController->download();
 
 		$this->assertInstanceOf('\OCP\AppFramework\Http\StreamResponse', $response);
+		$headers = $response->getHeaders();
+		$this->assertEquals('application/octet-stream', $headers['Content-Type']);
+		$this->assertEquals('attachment; filename="owncloud.log"', $headers['Content-Disposition']);
 	}
 }