Opened 8 years ago

Closed 8 years ago

#7902 closed (fixed)

/password_change/ is 404

Reported by: Thomas Kerpe Owned by: nobody
Component: *.djangoproject.com Version:
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

After a password reset via: http://www.djangoproject.com/accounts/password/reset/ I got an e-mail which contains a link to http://www.djangoproject.com/password_change/ which returns 404.
A working link would be http://www.djangoproject.com/accounts/password/change/

Attachments (2)

password_email_link.diff (579 bytes) - added by Michael Newman 8 years ago.
password_email_url.diff (611 bytes) - added by Bob Thomas <bthomas@…> 8 years ago.
Replace hardcoded link with url tag

Download all attachments as: .zip

Change History (8)

comment:1 Changed 8 years ago by Michael Newman

Component: Django Web siteAdmin interface
Has patch: set
milestone: 1.0
Triage Stage: UnreviewedAccepted

This is pretty straight forward, I added a patch for simplicity. This hard coded link should be replaced with a url tag, once that is possible.

Changed 8 years ago by Michael Newman

Attachment: password_email_link.diff added

comment:2 Changed 8 years ago by Bob Thomas <bthomas@…>

Cc: bthomas@… added

{% url django.contrib.auth.views.password_change %} works on my site.

Changed 8 years ago by Bob Thomas <bthomas@…>

Attachment: password_email_url.diff added

Replace hardcoded link with url tag

comment:3 Changed 8 years ago by Thomas Kerpe

Component: Admin interfaceDjango Web site
milestone: 1.0

The bug was reported against Django Web site not against admin, also admin is definitively wrong. Setting it back to Django Web site.

The template-problem is not in admin but just a missing password_reset_email.txt template in djangoproject website.

Bob Thomas approach using {% url django.contrib.auth.views.password_change %} in the contrib app template is ok and more generic.
An alternative would be to just add the corrected template to django_website/templates/registration/password_reset_email.txt

comment:4 Changed 8 years ago by Bob Thomas <bthomas@…>

This may be obsolete now, see ticket:8129

comment:5 Changed 8 years ago by Bob Thomas

Cc: bthomas@… removed

comment:6 Changed 8 years ago by Jacob

Resolution: fixed
Status: newclosed

(In [8925]) [djangoproject.com] a bunch of fixes to the website. Fixes #7271, #7459, #7902, #8271, #7949, #8845.

Note: See TracTickets for help on using tickets.
Back to Top