agw/overleaf-cep
1
0
Fork 0
mirror of https://github.com/yu-i-i/overleaf-cep.git synced 2026-11-15 14:43:46 +01:00
Code Issues Packages Projects Releases Wiki Activity

[misc] drop duplicate log line for unauthorized applyOtUpdate calls

Browse Source 
The violation is logged in Router._handleError.
This commit is contained in:
Jakob Ackermann
2020-08-27 10:22:31 +01:00
parent 425052ff91
commit 2ce7b36c95
2 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
services/real-time/app/js/WebsocketController.js
-4
View File
@@ -462,10 +462,6 @@ module.exports = WebsocketController = {
update, update,
function (error) { function (error) {
if (error) { if (error) {
logger.warn(
{ err: error, doc_id, client_id: client.id, version: update.v },
'client is not authorized to make update'
)
setTimeout( setTimeout(
() => () =>
// Disconnect, but give the client the chance to receive the error // Disconnect, but give the client the chance to receive the error
services/real-time/test/unit/js/WebsocketControllerTests.js
+2 -2
View File
@@ -1493,8 +1493,8 @@ describe('WebsocketController', function () {
// it "should disconnect the client", -> // it "should disconnect the client", ->
// @client.disconnect.called.should.equal true // @client.disconnect.called.should.equal true
it('should log a warning', function () { it('should not log a warning', function () {
return this.logger.warn.called.should.equal(true) return this.logger.warn.called.should.equal(false)
}) })
return it('should call the callback with the error', function () { return it('should call the callback with the error', function () {