Edgewall Software

Opened 4 years ago

Last modified 4 years ago

#12658 closed defect

RecipientMatcher prevents EmailResolver from working — at Version 4

Reported by: strk@… Owned by: Jun Omae
Priority: normal Milestone: 1.2.1
Component: notification Version: 1.2
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description (last modified by Jun Omae)

As reported on the users mailing list I could not succeed in installing a custom EmailResolver without changing the RecipientMatcher class to give callers a chance to lookup names.

I'm attaching a patch.

Change History (6)

by strk@…, 4 years ago

comment:1 by anonymous, 4 years ago

I've completed my first trac plugin implementation, which relies on this patch to be functional: https://git.osgeo.org/gogs/sac/TracLDAPEmailResolverPlugin

comment:2 by strk@…, 4 years ago

Milestone: 1.2.1

I hope this can be in 1.2.1, setting the milestone

comment:3 by Jun Omae, 4 years ago

Description: modified (diff)
Owner: set to Jun Omae
Status: newassigned

I consider suggested patch is incorrect. It should return (sid, 1, None) only when the user is listed in Environment.get_known_users().

comment:4 by Jun Omae, 4 years ago

Description: modified (diff)

by strk@…, 4 years ago

Modified as suggested

Note: See TracTickets for help on using tickets.