cargomedia / cm-janus

UNMAINTAINED. cm/janus bridge
MIT License
2 stars 6 forks source link

Reduce number of (debug) log events #232

Closed njam closed 8 years ago

njam commented 8 years ago

Currently 50'000 events per hour => ~14 events per second.

screen shot 2016-04-05 at 10 18 17

Ideas:

njam commented 8 years ago

As discussed with Tomasz earlier: Let's change the log verbosity to INFO on production, and make sure that it contains all relevant logs. Then for development we can keep DEBUG which might be useful.

Can you do the necessary changes?

vogdb commented 8 years ago

ok

njam commented 8 years ago

DEBUG: Combine messages that come from browser and go to janus.

INFO:

WARN: Errors on external services

ERROR: Unexpected internal and external errors

FATAL: Uncaught errors

vogdb commented 8 years ago

@tomaszdurka please review

tomaszdurka commented 8 years ago

Please merge upstream. Maybe make sense to redo the pr.

vogdb commented 8 years ago

@tomaszdurka please review. Please pay attention to how I moved logging from lib/connection.js to lib/proxy.js

tomaszdurka commented 8 years ago

Review done

vogdb commented 8 years ago

@tomaszdurka please review

tomaszdurka commented 8 years ago

lgtm