#5394 closed defect (fixed)
Wrong url for forgot_password in basic_auth plugin
| Reported by: | jerome | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.9.0 |
| Component: | programming | Keywords: | auth, views, routes |
| Cc: | Parent Tickets: |
Description
The endpoint used here does not exist and should surely be mediagoblin.plugins.basic_auth.forgot_password instead.
Attachments (1)
Change History (4)
by , 9 years ago
| Attachment: | fix-basic_auth-views.patch added |
|---|
comment:1 by , 9 years ago
| Keywords: | auth views routes added |
|---|---|
| Priority: | major → minor |
| Status: | new → review |
comment:2 by , 9 years ago
| Milestone: | → 0.9.0 |
|---|---|
| Resolution: | → fixed |
| Status: | review → closed |
Test coverage would be nice, but is not necessary. I pushed the patch!
jerome, do you have a preferred name you'd like to be credited with in the next release, or would you like to stick with "jerome"?
comment:3 by , 9 years ago
Sorry, I'm not really familiar with Trac and notifications yet, I just discovered your answers after reading your exciting article about the 0.9.0 release!... And realized that someone with the same first name than me was in the list of people to applause. I didn't except that with such a small contribution. So, thank you! I hope to contribute more in the future.
I take the opportunity to give a round of applause for your great work on MediaGoblin to you!!
End of the out of topic.
I just confirmed that this issue does exist, and that the attached patch does resolve the issue. Patch looks good to me.
This issue only pops up if you enter the name of a user which doesn't exist on the server, so I'm dropping the priority to minor. I'm also marking as ready for review, but I may go ahead and push if nobody else gets around to look at it.
It looks like we need test coverage here, but I'm not very familiar with creating unit tests.