The Address Book doesn't show the LDAP-field "labeledURI" as Website

RESOLVED FIXED in Thunderbird 3.0b4

Status

RESOLVED FIXED
14 years ago
10 years ago

People

(Reporter: bugzilla.mozilla.org, Assigned: pi)

Tracking

Trunk
Thunderbird 3.0b4

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 4 obsolete attachments)

(Reporter)

Description

14 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; de-DE; rv:1.7) Gecko/20040917 Firefox/0.9.3
Build Identifier: Mozilla Thunderbird 0.8 (20040913)

I have an LDAP server which contains some "inetOrgPerson" records. When I look
at them using Mozilla Thunderbird's Address Book it doesn't show die LDAP-field
"labeledURI". There's nothing shown in the field 'Website' in the Address Card.

Reproducible: Always
Steps to Reproduce:
1. Create new Directory Server entry to an LDAP server which contains
inetOrgPerson entries
2. Search for one of these entries where the field "labeledURI" is set
3. Open the entry
4. Click on "Address"
Actual Results:  
The two "Website" fields are empty.

Expected Results:  
The second "Website" field (the every last text field) should contain the value
of the field "labeledURI".

The schema "inetOrgPerson" as of RFC 2798 contains a field "labeledURI". Since
Thunderbird respects the other fields (cn, sn, givenName, st, title, ...) it
should respect the field "labeledURI" as well.

Comment 1

14 years ago
-> Core:MailNews:LDAP Integration
Assignee: mscott → sspitzer
Component: Address Book → MailNews: LDAP Integration
Product: Thunderbird → Core
QA Contact: grylchan
Version: unspecified → Trunk
This is an automated message, with ID "auto-resolve01".

This bug has had no comments for a long time. Statistically, we have found that
bug reports that have not been confirmed by a second user after three months are
highly unlikely to be the source of a fix to the code.

While your input is very important to us, our resources are limited and so we
are asking for your help in focussing our efforts. If you can still reproduce
this problem in the latest version of the product (see below for how to obtain a
copy) or, for feature requests, if it's not present in the latest version and
you still believe we should implement it, please visit the URL of this bug
(given at the top of this mail) and add a comment to that effect, giving more
reproduction information if you have it.

If it is not a problem any longer, you need take no action. If this bug is not
changed in any way in the next two weeks, it will be automatically resolved.
Thank you for your help in this matter.

The latest beta releases can be obtained from:
Firefox:     http://www.mozilla.org/projects/firefox/
Thunderbird: http://www.mozilla.org/products/thunderbird/releases/1.5beta1.html
Seamonkey:   http://www.mozilla.org/projects/seamonkey/
This bug has been automatically resolved after a period of inactivity (see above
comment). If anyone thinks this is incorrect, they should feel free to reopen it.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → EXPIRED
(Reporter)

Updated

13 years ago
Status: RESOLVED → UNCONFIRMED
OS: Windows 98 → All
Resolution: EXPIRED → ---
sorry for the spam.  making bugzilla reflect reality as I'm not working on these bugs.  filter on FOOBARCHEESE to remove these in bulk.
Assignee: sspitzer → nobody
(Assignee)

Comment 5

11 years ago
I can confirm this behavior using an LDAP server and a record that conforms to the inetOrgPerson schema.  There is more information about the labeledURI attribute in RFC 2079:
http://www.rfc-archive.org/getrfc.php?rfc=2079

Basically, it contains a URI and, optionally, a label separated from the URI by one or more spaces.

Before submitting a patch, should this URI be set as the work or home webpage?  I am leaning toward the work webpage, but either one would make sense.  And then, in which order should the other attributes for the URL appear in the default attribute map?
For example (mailnews/mailnews.js): pref("ldap_2.servers.default.attrmap.WebPage1", "mozillaWorkUrl,workurl,labeledURI");

My current patch removes the label because, if the label is kept, the link does not work as expected when viewing the card.

Assigning the bug to myself and marking as NEW...
Assignee: nobody → joshgeenen+bugzilla
Status: UNCONFIRMED → NEW
Ever confirmed: true

Updated

11 years ago
QA Contact: grylchan → ldap-integration
Product: Core → MailNews Core
(Assignee)

Comment 6

10 years ago
Created attachment 377899 [details] [diff] [review]
A patch for LDAP and LDIF imports

This patch adds labeledURI to the LDAP attribute map and LDIF imports under the Work URL and strips the label.  It worked for a basic LDIF import and on my local LDAP setup with and without a label.

I can write a quick unit test if this is worth fixing.
(Reporter)

Comment 7

10 years ago
Since the current implementation violates the LDAP standard, I do think this is "worth fixing".

It would be great if there was at least one free mail client which doesn't require nasty quirks to work with a totally standard conforming LDAP address book.
(In reply to comment #7)
> Since the current implementation violates the LDAP standard, I do think this is
> "worth fixing".
> 
WONTFIX ?
(Reporter)

Comment 9

10 years ago
(In reply to comment #8)
> (In reply to comment #7)
> > Since the current implementation violates the LDAP standard, I do think this is
> > "worth fixing".
> > 
> WONTFIX ?

No, that's not what I meant. To make it more clear:

The current implementation which ignores the labeledUri attribute violates the LDAP standard, so I would really love to see this fixed. The proposed patch is a step into the right direction. I think that it is worth the work to add additional unit tests and to include the patch.
(Assignee)

Comment 10

10 years ago
Created attachment 382054 [details] [diff] [review]
Updated path and unit test

This is an updated patch and unit test.  The unit test fails before the patch and passes after, as expected.
Attachment #377899 - Attachment is obsolete: true
(Assignee)

Comment 11

10 years ago
Created attachment 382073 [details] [diff] [review]
Patch and unit test

My last attachment did not contain the complete unit test (I forgot to include a the new LDIF used in the import).

The test passes in Windows and Linux with debugging disabled.
Attachment #382054 - Attachment is obsolete: true
Josh you needs to specify who should do the review. I don't know who does it for LDAP so I've cced a few persons that know better.
Hardware: x86 → All
(Assignee)

Comment 13

10 years ago
Comment on attachment 382073 [details] [diff] [review]
Patch and unit test

Let me know if this review should go to someone else.
Attachment #382073 - Flags: review?(bugzilla)
Comment on attachment 382073 [details] [diff] [review]
Patch and unit test

>diff -r 120b7786e2b2 mailnews/import/test/unit/resources/bug_263304.ldif

>+labeledURI: http://127.0.0.1

This should have a test label on the end, otherwise you're not actually testing the bug.

r=Standard8 with that fixed.
Attachment #382073 - Flags: review?(bugzilla) → review+
(Assignee)

Comment 15

10 years ago
Created attachment 389607 [details] [diff] [review]
Patch and unit test v2

Adds a label to the labeledURI in the imported LDIF file as requested.
Attachment #382073 - Attachment is obsolete: true
Attachment #389607 - Flags: review+
(Assignee)

Updated

10 years ago
Attachment #389607 - Flags: superreview?(bienvenu)

Comment 16

10 years ago
Comment on attachment 389607 [details] [diff] [review]
Patch and unit test v2

use -1 instead of kNotFound here, since kNotFound should be going away:

+      if (index != kNotFound)
+        column.Truncate(index);
Attachment #389607 - Flags: superreview?(bienvenu) → superreview+
(Assignee)

Comment 17

10 years ago
Created attachment 392038 [details] [diff] [review]
Patch & unit test for checkin

This patch changes the kNotFound to -1.

Thanks for the r and sr.
Attachment #389607 - Attachment is obsolete: true
Attachment #392038 - Flags: superreview+
Attachment #392038 - Flags: review+
(Assignee)

Updated

10 years ago
Status: NEW → ASSIGNED
Keywords: checkin-needed
Checked in: http://hg.mozilla.org/comm-central/rev/f7e7acfc8080
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.0b4
You need to log in before you can comment on or make changes to this bug.