Opened 17 years ago

Closed 17 years ago

Last modified 17 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: dev
Severity: Keywords:
Cc: richard.davies@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

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 17 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 by Jacob, 17 years ago

Has patch: unset
Triage Stage: Unreviewed → Accepted

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

by Dave Naffziger, 17 years ago

Attachment: new_email.diff​ added

comment:2 by Matt McClanahan, 17 years ago

Has patch: set

comment:3 by Matt McClanahan, 17 years ago

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

comment:4 by Russell Keith-Magee, 17 years ago

Resolution: → fixed
Status: new → closed

(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 by Richard Davies <richard.davies@…>, 17 years ago

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