Skip to content

Commit

Permalink
Merge pull request #2322 from nextcloud/compat-with-chrome54
Browse files Browse the repository at this point in the history
Remove exception for Chrome on Mobile
  • Loading branch information
MorrisJobke committed Nov 25, 2016
2 parents e502a2d + 2144a11 commit 4764679
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions lib/base.php
Original file line number Diff line number Diff line change
Expand Up @@ -538,15 +538,6 @@ private static function performSameSiteCookieProtection() {
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();
$processingScript = explode('/', $requestUri);
Expand Down

0 comments on commit 4764679

Please sign in to comment.