瀏覽代碼

fix(JitsiMeetLogStorage): do not log whole message

The logMessage can be very long. Probably only it's length could be
important to why sendApplicationLog has failed. Stringify the error
though.
j8
paweldomas 7 年之前
父節點
當前提交
3b6e34e96b
共有 1 個檔案被更改,包括 2 行新增2 行删除
  1. 2
    2
      react/features/base/logging/JitsiMeetLogStorage.js

+ 2
- 2
react/features/base/logging/JitsiMeetLogStorage.js 查看文件

81
         try {
81
         try {
82
             conference.sendApplicationLog(logMessage);
82
             conference.sendApplicationLog(logMessage);
83
         } catch (error) {
83
         } catch (error) {
84
-            // FIXME whole objects logged
85
             // NOTE console is intentional here
84
             // NOTE console is intentional here
86
             console.error(
85
             console.error(
87
-                'Failed to store the logs: ', logMessage, error);
86
+                `Failed to store the logs, msg length: ${logMessage.length}`
87
+                    + `error: ${JSON.stringify(error)}`);
88
         }
88
         }
89
     }
89
     }
90
 }
90
 }

Loading…
取消
儲存