From 1fcf534dcfcb161c92321b2f4730a2921a4af934 Mon Sep 17 00:00:00 2001
From: Jakob Ackermann <jakob.ackermann@overleaf.com>
Date: Wed, 22 Apr 2020 11:36:20 +0200
Subject: [PATCH] [Router] revert preserving of client.ol_context

We do not enter this line twice, it would result in multiple event
 handlers too.
---
 services/real-time/app/coffee/Router.coffee | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/services/real-time/app/coffee/Router.coffee b/services/real-time/app/coffee/Router.coffee
index b7d1ae2662..3d891f1476 100644
--- a/services/real-time/app/coffee/Router.coffee
+++ b/services/real-time/app/coffee/Router.coffee
@@ -55,7 +55,9 @@ module.exports = Router =
 		app.post "/client/:client_id/disconnect", httpAuth, HttpApiController.disconnectClient
 
 		session.on 'connection', (error, client, session) ->
-			client.ol_context = {} unless client.ol_context
+			# init client context, we may access it in Router._handleError before
+			#  setting any values
+			client.ol_context = {}
 
 			client?.on "error", (err) ->
 				logger.err { clientErr: err }, "socket.io client error"