Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#8129 closed (fixed)

Modify password reset email template to allow user configured urls

Reported by: Dave Naffziger Owned by: nobody
Component: contrib.admin Version: master
Severity: Keywords:
Cc: richard.davies@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


This line is currently used to generate password reset urls:

{{ protocol }}://{{ domain }}/reset/{{ uid }}-{{ token }}/

This creates a fixed url in every email sent, so the user cannot customize the URL.

The proposed change is to use this line instead:

{{ protocol }}://{{ domain }}{% url django.contrib.auth.views.password_reset_confirm uidb36=uid, token=token %}

This is a single line patch so I won't upload diff files, etc.

Attachments (1)

new_email.diff (691 bytes) - added by Dave Naffziger 8 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 8 years ago by Jacob

Has patch: unset
Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedAccepted

Please do upload a diff; it makes the whole process quicker.

Changed 8 years ago by Dave Naffziger

Attachment: new_email.diff added

comment:2 Changed 8 years ago by Matt McClanahan

Has patch: set

comment:3 Changed 8 years ago by Matt McClanahan

#8451 is a dup, but also has a patch.

comment:4 Changed 8 years ago by Russell Keith-Magee

Resolution: fixed
Status: newclosed

(In [8517]) Fixed #8129: Modified the password reset email template to use a URL lookup, so that the email doesn't require customization based on the way that the reset view is deployed. Thanks to davenaff for the report and fix.

comment:5 Changed 8 years ago by Richard Davies <richard.davies@…>

Cc: richard.davies@… added
Note: See TracTickets for help on using tickets.
Back to Top