459860 - Syncing passwords produces warnings concerning Log4Moz

Status: RESOLVED FIXED

(Cloud Services :: General, defect, P2)

Description

[Markus Hoofe]

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; de; rv:1.9.0.3) Gecko/2008092700 SUSE/3.0.3-2.2 Firefox/3.0.3
Build Identifier: 

Using the current source build (0.2.6 2008/10/14) the password synchronization produces to the following warning in the logs:

Service.Main    WARN    Engine exception: ReferenceError: Log4Moz is not defined

Since it is only a warning, i guess it's not tragic. But i had a look in password.js and found out that not only Log4Moz is NOT defined in the imports, but others (Cc, Cu etc.) are missing as well. So i don't really know if it's just a normal bug or even a critical one.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.

Comment 1

[Markus Hoofe]

Attachment: Patch for passwords.js

I could fix the warnings with this attached patch for passwords.js. I simply needed to add some imports. Hope i could help.

Comment 2

[Dan Mills [:thunder]]

Thanks!
We'll be re-writing password sync soon for 0.3; and will add these fixes then.

Comment 3

[Anant Narayanan [:anant]]

Fixed in 0.3 Password Sync