Using global LDAP server prefs for autocomplete lookup does not always work correctly

RESOLVED FIXED

Status

MailNews Core
Address Book
RESOLVED FIXED
12 years ago
9 years ago

People

(Reporter: Andreas Höfler, Assigned: standard8)

Tracking

({fixed-seamonkey1.1a, fixed1.8.1})

Trunk
fixed-seamonkey1.1a, fixed1.8.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 4 obsolete attachments)

(Reporter)

Description

12 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8) Gecko/20051111 Firefox/1.5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8) Gecko/20051111 Firefox/1.5

I created a LDAP Directory entry in Address Book and set it to be used in
Options->Composition->Addressing->Directory Server

Then I created a new account and made sure that
Account Settings->Composition & Addressing->Use my global LDAP server preferences for this account
was checked.

When trying to let TB autocomplete an address when writing a new mail, it was not found.

Then I choosed 
Account Settings->Composition & Addressing->Use a different LDAP server
and selected the one created in Address Book.

Trying again to let TB autocomplete an incomplete address in the compose window works now.

Reproducible: Always

Steps to Reproduce:
1. Create a LDAP directory in Address Book
2. Set global settings to include LDAP server in autocomplete search
3. Try to let Thunderbird autocomplete an address in the compose-window

Actual Results:  
Autocompletion does not work.

Expected Results:  
Address should be auto-completed.

Thunderbird version 1.5 Beta 2 (20051006), Mail Account lying on Exchange Server and is accessed through IMAP.

Comment 1

12 years ago
Reporter, can you confirm your LDAP settings?  Remember ActiveDirectory works on a different port.

But I am also having difficulties connecting to my active directory.
Hardware: PC → All
(Reporter)

Comment 2

12 years ago
I did not change the LDAP settings during the procedure, so I doubt to be the error hidden there somewhere.

I created the Address Book entry with:
Port Number: 3268
Base DN: CN=Users,DC=my,DC=company,DC=example
Search Filter: (objectclass=user)

Comment 3

12 years ago
Confirming.  I see this behaviour in Windows XP
version 1.6a1 (20051122)
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows 2000 → All
Version: unspecified → Trunk

Updated

12 years ago
Summary: Using globlal LDAP server prefs for account does not work with ActiveDirectory → Using global LDAP server prefs for account does not work with Active Directory

Comment 4

12 years ago
IIRC, ActiveDirectory doesn't include inetOrgPerson by default, and many of the fields that TB looks for by default are based on that.  You might try installing the inetOrgPerson piece on the server, and, failing that, it would be interesting to see a packet trace from Ethereal.
(Reporter)

Comment 5

12 years ago
Dan, I don't that my problems are originated in a communication problem with Active Directory but rather Thunderbird not using the global settings when told to do so.

As you can see in the initial report, autocompletion DOES work for me, but only if I turn it on explicitely on the account preferences. If I leave the settings to use the global settings, where the LDAP server is configured correctly, autocompletion seems to not query LDAP at all. The half-completed address turns red too fast for my taste, but this could be wrong as I'm not allowed to install a packet tracer on this computer.

Comment 6

12 years ago
I 've the same problem as descibed by Andreas.
Thunderbird version 1.5 RC1 (20051025)
(Assignee)

Comment 7

12 years ago
In debugging this, it would be useful to know if the value of the pref ldap_2.autoComplete.directoryServer is blank (default) or filled in.

Also, how many ldap servers have you set up? (I'm guessing just one).

Comment 8

12 years ago
(In reply to comment #7)
I've only one ldap server configured.

in prefs.js:
ldap_2.autoComplete.directoryServer is not set at all
but on screen there is one, that is configured

for autoComplete in prefs.js is just:
user_pref("ldap_2.autoComplete.useDirectory", true);

(Reporter)

Comment 9

12 years ago
I have only one set up too.

But at the moment I'm unable to reproduce my initial problem (altough I did the same procedure before).

When I re-set the account setting to use the global settings I cannot make the bug re-appear and NOT perform auto-completion... strange.

Comment 10

12 years ago
My configuration of Thunderbird is migrated from mozilla rv:1.8a5 Gecko/20041122.
After migration ldap_2.autoComplete.directoryServer is not set at all.

When You go to Options/Composition/Adressing and do:
1. Click on dropdown 'Directory Server'
2. choose Server from list (only one position in my case)
3. Clik OK
then ldap_2.autoComplete.directoryServer is to proper value and autocompletion using global LDAP works.
(Assignee)

Comment 11

12 years ago
Ok, I'm seeing this as well on SeaMonkey. if ldap_2.autoComplete.directoryServer isn't set auto complete doesn't work (unfortunately it can be a right pain to get it to set in new profiles as well). Moving to core/address book as its a better location for this bug.

Patch coming up in a while hopefully...
Assignee: mscott → bugzilla
Component: Preferences → MailNews: Address Book
Product: Thunderbird → Core
QA Contact: preferences → addressbook
(Assignee)

Comment 12

12 years ago
Created attachment 204869 [details] [diff] [review]
The fix

This patch fixes the problem by forcing setting of the pref when it isn't set previously and there is more than one item in the ldap list. Its not necessarily ideal as preferably there should be proper setting from the menu popup - but as I'm also currently working on revising the whole ldap autocomplete menus & backend code elsewhere, it should do as an intermediate fix.
Attachment #204869 - Flags: superreview?(bienvenu)
Attachment #204869 - Flags: review?(neil.parkwaycc.co.uk)

Comment 13

12 years ago
Comment on attachment 204869 [details] [diff] [review]
The fix

ok by me...
Attachment #204869 - Flags: superreview?(bienvenu) → superreview+

Comment 14

12 years ago
Comment on attachment 204869 [details] [diff] [review]
The fix

>-        if (!directoriesList.selectedItem)
>-          directoriesList.value = "";
This bit is necessary if all else fails, e.g. in the case of deleting the last directory.

>+        if (directoriesList.selectedIndex == -1 &&
Prefer !directoriesList.selectedItem (selectedIndex has to trawl through the childNodes looking for the item).

>+            gAvailDirectories.length >= 1 &&
Nit: != 0

>+            !gPrefInt.prefIsLocked("ldap_2.autoComplete.directoryServer"))
Nit: pref_string_title has this value (oddly, if the declaration was moved up a bit then it could get reused a third time).
Attachment #204869 - Flags: review?(neil.parkwaycc.co.uk) → review-

Comment 15

12 years ago
How can I determine which nightly build to download to test this fix?  I have to admit I'm new to the mozilla processes, so I apologize if this is a FAQ.
(Assignee)

Comment 16

12 years ago
(In reply to comment #15)
> How can I determine which nightly build to download to test this fix?  I have
> to admit I'm new to the mozilla processes, so I apologize if this is a FAQ.
> 

There is no point at the moment, the patch hasn't been granted review & hasn't been checked in (a note will be added when it is - or it'll be marked fixed). When it has been fixed/checked in wait a day or so & pick up the latest nightly.
I'm seeing this problem also and there are scattered dupes going back, some of which get closed WFM. Bug 312115 is one example. LDAP works if I choose a server in a particular account's addressing prefs, but not if I set that account to use the global option.

Where is the global option set? I've been assuming the addressing sub-tab of the composition tab on the Global options dialog. "When addressing messages, look for matching entries in:" and I have both Local Address Books and Directory Server checked, and the directory server is set to the same entry that works if I explicitly select it in each Account.
(Assignee)

Comment 18

12 years ago
(In reply to comment #17)
> Where is the global option set? I've been assuming the addressing sub-tab of
> the composition tab on the Global options dialog. "When addressing messages,
> look for matching entries in:" and I have both Local Address Books and
> Directory Server checked, and the directory server is set to the same entry
> that works if I explicitly select it in each Account.

That is correct - sometimes, especially with just one ldap server setup, although the global options dialog displays the address book, the pref is actually "", and may never get set correctly. If you play around a bit you can sometimes get it to work (especially with > 1 ldap address book) and thats probably why the bugs have been marked as WFM.
(Assignee)

Comment 19

12 years ago
Created attachment 205993 [details] [diff] [review]
Revised patch

This is a revised version. After looking at Neil's comments from the previous patch I realised that the case where you add a ldap directory via edit directories wasn't working correctly, so I looked at fixing that as well.

As it turns out the easiest way of fixing this in the short term (still bearing in mind I want to rewrite a lot of the backend for ldap prefs) is to add the "None" option that is currently in the Mail/Newsgroup account settings autocomplete override preferences to the global preference dialog as well. This simplifies the code a bit as well.
Attachment #204869 - Attachment is obsolete: true
Attachment #205993 - Flags: superreview?(bienvenu)
Attachment #205993 - Flags: review?(neil.parkwaycc.co.uk)
(Assignee)

Comment 20

12 years ago
Changing summary to better reflect the problem

was: Using global LDAP server prefs for account does not work with Active Directory

now: Using global LDAP server prefs for autocomplete lookup does not always work correctly
Summary: Using global LDAP server prefs for account does not work with Active Directory → Using global LDAP server prefs for autocomplete lookup does not always work correctly

Updated

12 years ago
Attachment #205993 - Flags: superreview?(bienvenu) → superreview+
(Assignee)

Comment 21

12 years ago
*** Bug 299968 has been marked as a duplicate of this bug. ***
(Assignee)

Comment 22

12 years ago
Comment on attachment 205993 [details] [diff] [review]
Revised patch

The patch is still ok, but I just realised that thunderbird uses this code as well.
Attachment #205993 - Attachment description: Revised patch (SeaMonkey verison) → Revised patch
(Assignee)

Comment 23

12 years ago
Comment on attachment 205993 [details] [diff] [review]
Revised patch

Cancelling review request for the time being, I want to come up with a full patch for both SeaMonkey & Thunderbird
Attachment #205993 - Flags: review?(neil.parkwaycc.co.uk)

Comment 24

12 years ago
*** Bug 316766 has been marked as a duplicate of this bug. ***

Comment 25

12 years ago
Interestingly enough, my global ldap server has started working properly in the last week or so.  I'm running 1.5 build 20051201, and haven't upgraded in 3-4 weeks, so this is something of a mystery, but I'm a happy user :)
(Assignee)

Comment 26

12 years ago
Created attachment 207846 [details] [diff] [review]
Patch v3

Revised patch for both SeaMonkey & Thunderbird, same principles as before, but I've also removed the gRefresh check which will mean we always ensure a selection even in the worst cases.
Attachment #205993 - Attachment is obsolete: true
Attachment #207846 - Flags: review?(neil.parkwaycc.co.uk)

Comment 27

12 years ago
Comment on attachment 207846 [details] [diff] [review]
Patch v3

Preferences/Addressing
Create two LDAP entries
Select the second entry
Delete it
Result: disabled menulist :-(
Attachment #207846 - Flags: review?(neil.parkwaycc.co.uk) → review-
(Assignee)

Comment 28

12 years ago
Created attachment 207890 [details] [diff] [review]
Patch v4

Good catch Neil, this version checks how many directories there are before disabling the menu.
Attachment #207846 - Attachment is obsolete: true
Attachment #207890 - Flags: review?(neil.parkwaycc.co.uk)

Comment 29

12 years ago
Comment on attachment 207890 [details] [diff] [review]
Patch v4

Actually, I no longer see why it's necessary to disable the menulist (except via normal pref locking), so we could elimintate gFromGlobalPref.
Attachment #207890 - Flags: review?(neil.parkwaycc.co.uk) → review+
(Assignee)

Comment 30

12 years ago
Created attachment 208307 [details] [diff] [review]
Patch v5 (checked in to trunk and branch)

Update patch to address Neil's comment and remove gFromGlobalPref. Carrying forward his r.
Attachment #208307 - Flags: superreview?(bienvenu)
Attachment #208307 - Flags: review+

Updated

12 years ago
Attachment #208307 - Flags: superreview?(bienvenu) → superreview+
(Assignee)

Updated

12 years ago
Attachment #207890 - Attachment is obsolete: true
(Assignee)

Comment 31

12 years ago
Comment on attachment 208307 [details] [diff] [review]
Patch v5 (checked in to trunk and branch)

Patch checked in to trunk.

Requesting approval for 1.8.1 branch. This is a Thunderbird and SeaMonkey patch that corrects a problem with preferences where we indicate that the user has selected a directory for use in autocomplete, but the preference hasn't actually been set.
Attachment #208307 - Attachment description: Patch v5 → Patch v5 (checked in to trunk)
Attachment #208307 - Flags: approval1.8.1?
(Assignee)

Comment 32

12 years ago
Fix is in on trunk -> Fixed. I'll check into the 1.8.1 branch when the patch eventually gets approved.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Updated

12 years ago
Attachment #208307 - Flags: approval1.8.1? → branch-1.8.1?(mscott)

Updated

12 years ago
Attachment #208307 - Flags: approval-branch-1.8.1?(mscott) → approval-branch-1.8.1+
(Assignee)

Updated

12 years ago
Keywords: fixed-seamonkey1.1a, fixed1.8.1
(Assignee)

Updated

12 years ago
Attachment #208307 - Attachment description: Patch v5 (checked in to trunk) → Patch v5 (checked in to trunk and branch)
(Assignee)

Updated

10 years ago
Duplicate of this bug: 299968
(Assignee)

Updated

9 years ago
Duplicate of this bug: 323655
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.