[Bug 837700] [NEW] Issues with users subscribed via their preferred address

Public bug reported: You can subscribe an address to a mailing list, and you can subscribe a user to a mailing list, if the user has a preferred address. There are two problems with this. * You can actually end up getting doubly subscribed (i.e. same address with the same role). Normally, the subscription mechanism prevents you from subscribing the same address with the same role more than once, or the same user with the same role more than once. However, if you subscribe once by the address, and once by the user whose preferred address is that same address, the double subscription occurs. * The Member repr() doesn't distinguish between a member that's subscribed via explicit address, and one that's subscribed via preferred address. This means that the above will appear as a double address subscription, when it's really not. Are these substantial problems that need to be fixed? Almost certainly the second one does, but I haven't come up with an elegant repr() for the case of a user subscribed with their preferred address. ** Affects: mailman Importance: Low Status: Confirmed ** Tags: mailman3 -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/837700 Title: Issues with users subscribed via their preferred address To manage notifications about this bug go to: https://bugs.launchpad.net/mailman/+bug/837700/+subscriptions

Some thoughts: First, this should not cause problems with multiple deliveries since MM creates a set of recipient addresses for the message, which filters out duplicates. As for Member.__repr__() perhaps something like this (untested): def __repr__(self): if self._address is None: subscription = 'preferred={0}'.format(self._user.preferred_address) else: subscription = self._address return '<Member: {0} on {1} as {2}>'.format( subscription, self.mailing_list, self.role) Can you try that and see what you think? As for #1, I'm not sure it's a problem given the above, but if it does prove to be so, I can think of a few alternatives. Maybe the user should be disallowed from subscribing when an address they control is already subscribed with the same role. It's theoretically possible that an address could come under the control of the user after the fact, and that would be difficult to catch. Another possibility would be to convert the existing address-based subscription to a user-based subscription, but that might clobber the address that they actually want to get delivery on. E.g. they're subscribed with anne@example.com but their preferred is aperson@example.com. Any other ideas? I think if this is a problem, in the face of ambiguity, refuse to guess (i.e. disallow those double subscriptions). -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/837700 Title: Issues with users subscribed via their preferred address To manage notifications about this bug go to: https://bugs.launchpad.net/mailman/+bug/837700/+subscriptions

This bug has been moved to the new gitlab repo here: https://gitlab.com/mailman/mailman/issues/29 -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/837700 Title: Issues with users subscribed via their preferred address To manage notifications about this bug go to: https://bugs.launchpad.net/mailman/+bug/837700/+subscriptions

Some thoughts: First, this should not cause problems with multiple deliveries since MM creates a set of recipient addresses for the message, which filters out duplicates. As for Member.__repr__() perhaps something like this (untested): def __repr__(self): if self._address is None: subscription = 'preferred={0}'.format(self._user.preferred_address) else: subscription = self._address return '<Member: {0} on {1} as {2}>'.format( subscription, self.mailing_list, self.role) Can you try that and see what you think? As for #1, I'm not sure it's a problem given the above, but if it does prove to be so, I can think of a few alternatives. Maybe the user should be disallowed from subscribing when an address they control is already subscribed with the same role. It's theoretically possible that an address could come under the control of the user after the fact, and that would be difficult to catch. Another possibility would be to convert the existing address-based subscription to a user-based subscription, but that might clobber the address that they actually want to get delivery on. E.g. they're subscribed with anne@example.com but their preferred is aperson@example.com. Any other ideas? I think if this is a problem, in the face of ambiguity, refuse to guess (i.e. disallow those double subscriptions). -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/837700 Title: Issues with users subscribed via their preferred address To manage notifications about this bug go to: https://bugs.launchpad.net/mailman/+bug/837700/+subscriptions

This bug has been moved to the new gitlab repo here: https://gitlab.com/mailman/mailman/issues/29 -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/837700 Title: Issues with users subscribed via their preferred address To manage notifications about this bug go to: https://bugs.launchpad.net/mailman/+bug/837700/+subscriptions
participants (2)
-
Abhilash Raj
-
Barry Warsaw