Browse Source

improve logging by avoiding string escapes

Brian Gough 10 years ago
parent
commit
ba0de92081
1 changed files with 8 additions and 6 deletions
  1. 8 6
      services/track-changes/app.coffee

+ 8 - 6
services/track-changes/app.coffee

@@ -7,16 +7,18 @@ if Settings.sentry?.dsn?
 
 
 # log updates as truncated strings
 # log updates as truncated strings
 truncateFn = (updates) ->
 truncateFn = (updates) ->
-		JSON.stringify updates, (key, value) ->
-			if typeof value == 'string' && (len = value.length) > 80
-				return value.substr(0,32) + "...(message of length #{len} truncated)..." + value.substr(-32)
-			else
-				return value
+		JSON.parse(
+			JSON.stringify updates, (key, value) ->
+				if typeof value == 'string' && (len = value.length) > 80
+					return value.substr(0,32) + "...(message of length #{len} truncated)..." + value.substr(-32)
+				else
+					return value
+		)
 
 
 TrackChangesLogger.addSerializers {
 TrackChangesLogger.addSerializers {
+	rawUpdate: truncateFn
 	rawUpdates: truncateFn
 	rawUpdates: truncateFn
 	newUpdates: truncateFn
 	newUpdates: truncateFn
-	rawUpdate: truncateFn
 	lastUpdate: truncateFn
 	lastUpdate: truncateFn
 }
 }