diff options
| author | Lukas Reschke <lukas@statuscode.ch> | 2016-09-20 11:08:32 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2016-09-20 11:08:32 +0200 |
| commit | 09e1218df939ffb28160d054bed28af8fc29113e (patch) | |
| tree | 333709ebe1fa49c31b0a3ab3a3feffecf53061d6 | |
| parent | 70660d0d6c789735e27b7faf301ad9ccc015d1df (diff) | |
| parent | 9a2511fe9bd5716c49bc8a28cbea690f9bb0fe97 (diff) | |
| download | nextcloud-server-09e1218df939ffb28160d054bed28af8fc29113e.tar.gz nextcloud-server-09e1218df939ffb28160d054bed28af8fc29113e.zip | |
Merge pull request #1454 from nextcloud/morris
Don't check Same-Site attribute for mobile chrome
| -rw-r--r-- | lib/base.php | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/lib/base.php b/lib/base.php index b2e45595a3e..7d86245818d 100644 --- a/lib/base.php +++ b/lib/base.php @@ -520,6 +520,14 @@ class OC { return; } + // Chrome on Android has a bug that it doesn't sent cookies with the + // same-site attribute for the download manager. To work around that + // all same-site cookies get deleted and recreated directly. Awesome! + // FIXME: Remove once Chrome 54 is deployed to end-users + // @see https://github.com/nextcloud/server/pull/1454 + if($request->isUserAgent([\OC\AppFramework\Http\Request::USER_AGENT_ANDROID_MOBILE_CHROME])) { + return; + } if(count($_COOKIE) > 0) { $requestUri = $request->getScriptName(); |