Skip to content

Commit

Permalink
fix: Set expect guzzle option to false
Browse files Browse the repository at this point in the history
This option can be removed once Collabora Online has proper support for
the Expect: 100-Continue header - until then this will avoid issues with
larger file previews

Signed-off-by: Julius Härtl <[email protected]>
  • Loading branch information
juliusknorr authored and backportbot-nextcloud[bot] committed Nov 16, 2023
1 parent ec432a4 commit 5d4f82f
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion lib/Preview/Office.php
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,11 @@ public function getThumbnail($path, $maxX, $maxY, $scalingup, $fileview) {
}

$client = $this->clientService->newClient();
$options = ['timeout' => 25];
$options = [
'timeout' => 25,
// FIXME: Can be removed once https://github.com/CollaboraOnline/online/issues/6983 is fixed upstream
'expect' => false,
];

if ($this->config->getAppValue('richdocuments', 'disable_certificate_verification') === 'yes') {
$options['verify'] = false;
Expand Down

0 comments on commit 5d4f82f

Please sign in to comment.