fix link to held messages in "requires approval" email

Bug #1444493 reported by Sumana Harihareswara
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Postorius
New
Undecided
Abhilash Raj

Bug Description

A nonmember tried to post to a list. The email notification should have pointed to

http://irc.asynchronous.in/postorius/lists/pycon-sprint.mailman.asynchronous.in/held_messages

but instead the link went to

    http://<email address hidden>

Revision history for this message
Sumana Harihareswara (sumanah) wrote :

http://<email address hidden>/d13f1aaffbe174d449f38cb2d33874ff788b4852 -- the "cancel" link also doesn't work:

Page not found (404)
Request Method: GET
Request URL: http://<email address hidden>/d13f1aaffbe174d449f38cb2d33874ff788b4852

Using the URLconf defined in urls, Django tried these URL patterns, in this order:

    ^browserid/login/$ [name='browserid.login']
    ^browserid/logout/$ [name='browserid.logout']
    ^browserid/csrf/$ [name='browserid.csrf']
    ^$
    ^postorius/

The current URL, <email address hidden>/d13f1aaffbe174d449f38cb2d33874ff788b4852, didn't match any of these.

You're seeing this error because you have DEBUG = True in your Django settings file. Change that to False, and Django will display a standard 404 page.

Revision history for this message
Sumana Harihareswara (sumanah) wrote :

Same problem for the "please confirm your subscription" email's link:

Page not found (404)
Request Method: GET
Request URL: http://mailman.asynchronous.in/confirm/f8a665fb4fffc1ff7289f598d6a4ee656679a387

Using the URLconf defined in urls, Django tried these URL patterns, in this order:

    ^browserid/login/$ [name='browserid.login']
    ^browserid/logout/$ [name='browserid.logout']
    ^browserid/csrf/$ [name='browserid.csrf']
    ^$
    ^postorius/

The current URL, confirm/f8a665fb4fffc1ff7289f598d6a4ee656679a387, didn't match any of these.

Revision history for this message
Sumana Harihareswara (sumanah) wrote :

These errors are still happening as of r235 of Postorius.

tags: added: mailman3
Changed in postorius:
assignee: nobody → Abhilash Raj (raj-abhilash1)
Revision history for this message
Abhilash Raj (raj-abhilash1) wrote :

Its a tricky process to actually update the urls, it would need plumbing things through REST and changes in core as well as postorius. Removing the url for now.

Revision history for this message
Abhilash Raj (raj-abhilash1) wrote :

This bug has been moved to the new gitlab repo here: https://gitlab.com/mailman/postorius/issues/16

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.