Open Bug 1801091 Opened 2 years ago

Figure out how to replace Cu.reportError calls in Log.sys.mjs

Categories

(Toolkit :: General, task, P3)

task

Tracking

()

People

(Reporter: standard8, Unassigned)

References

(Blocks 1 open bug)

Details

In bug 877389 we are replacing Cu.reportError with console.error.

For Log.sys.mjs, we need to figure out what to do. There are two places in that file that currently use Cu.reportError.

The first one in dumpError function can probably be completely replaced by console.error.

The second one is harder - Log.sys.mjs has both console and standard out options as separate appenders. Cu.reportError only logs to the console, so replacing it with console.error may cause duplicate logging which could be confusing.

We may be able to remove Log.sys.mjs entirely, but a few components (especially services) still have extensive use of it.

An intermediate alternative would be to see if we the consumers of Log.sys.mjs could log all messages to both console & stdout, hence combining the appenders.

You need to log in before you can comment on or make changes to this bug.