From b50b0c1e91b89ca1dc7f761c1f603722a136e3a3 Mon Sep 17 00:00:00 2001 From: andreas-bulling Date: Mon, 22 Feb 2021 09:35:40 +0100 Subject: [PATCH 1/2] Update docker-compose.yml I ran into the problem that Sharelatex would not connect to Redis. This fixes it (see https://github.com/overleaf/overleaf/pull/737) --- docker-compose.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docker-compose.yml b/docker-compose.yml index ac2a1c3..8a86d91 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -88,7 +88,7 @@ services: redis: restart: always - image: redis + image: redis:5.0.0 container_name: redis # modify to get rid of the redis issue #35 and #19 with a better solution # WARNING: /proc/sys/net/core/somaxconn is set to the lower value of 128. From abf1e765555ec7b0f2eec3c2b189c739c05e173c Mon Sep 17 00:00:00 2001 From: andreas-bulling Date: Mon, 22 Feb 2021 09:41:18 +0100 Subject: [PATCH 2/2] Update AuthenticationManager.js userId contains the whole user object. Hence, only use _id --- ldap-overleaf-sl/sharelatex/AuthenticationManager.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ldap-overleaf-sl/sharelatex/AuthenticationManager.js b/ldap-overleaf-sl/sharelatex/AuthenticationManager.js index ba7ba06..e5213e9 100644 --- a/ldap-overleaf-sl/sharelatex/AuthenticationManager.js +++ b/ldap-overleaf-sl/sharelatex/AuthenticationManager.js @@ -212,7 +212,7 @@ const AuthenticationManager = { } db.users.update( { - _id: ObjectId(userId.toString()) + _id: ObjectId(userId._id.toString()) }, { $set: {