Просмотр исходного кода

Merge pull request #78 from overleaf/spd-such-logs

Don't log document lines in log messages
Simon Detheridge 7 лет назад
Родитель
Сommit
ad485709cf

+ 3 - 2
services/document-updater/app.coffee

@@ -6,8 +6,9 @@ http = require("http")
 Settings = require('settings-sharelatex')
 logger = require('logger-sharelatex')
 logger.initialize("document-updater")
-logger.logger.serializers.docs = require("./app/js/LoggerSerializers").docs
-logger.logger.serializers.files = require("./app/js/LoggerSerializers").files
+
+logger.logger.addSerializers(require("./app/js/LoggerSerializers"))
+
 if Settings.sentry?.dsn?
 	logger.initializeErrorReporting(Settings.sentry.dsn)
 

+ 9 - 13
services/document-updater/app/coffee/LoggerSerializers.coffee

@@ -1,14 +1,10 @@
-module.exports =
-	docs: (docs) ->
-		docs.map (doc) ->
-			{
-				path: doc.path
-				id: doc.doc
-			}
+showLength = (thing) ->
+	"length: #{thing?.length}"
 
-	files: (files) ->
-		files.map (file) ->
-			{
-				path: file.path
-				id: file.file
-			}
+module.exports =
+	# replace long values with their length
+	lines: showLength
+	oldLines: showLength
+	newLines: showLength
+	ranges: showLength
+	update: showLength