#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: |
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)
Change History (6)
comment:1 Changed 10 years ago by
Has patch: | unset |
---|---|
Triage Stage: | Unreviewed → Accepted |
Changed 10 years ago by
Attachment: | new_email.diff added |
---|
comment:2 Changed 10 years ago by
Has patch: | set |
---|
comment:4 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:5 Changed 10 years ago by
Cc: | richard.davies@… added |
---|
Note: See
TracTickets for help on using
tickets.
Please do upload a diff; it makes the whole process quicker.