Closed
Bug 566907
Opened 15 years ago
Closed 15 years ago
Migrate user passwords to Kitsune when logging in to Tiki
Categories
(support.mozilla.org :: General, defect)
support.mozilla.org
General
Tracking
(Not tracked)
RESOLVED
FIXED
1.5.5
People
(Reporter: paulc, Assigned: paulc)
References
Details
(Whiteboard: [qa-])
Attachments
(1 file)
|
3.79 KB,
patch
|
jsocol
:
review+
|
Details | Diff | Splinter Review |
In 2.0 we've been creating Django users with empty passwords. For 2.1, however, we need to have users be able to log in to the admin area.
The best way to do this is to migrate the user's password on login - the only time we have it in plain text. This will be done on the Tiki side.
| Assignee | ||
Updated•15 years ago
|
Summary: [k] Migrate user passwords to Kitsune when logging in to Tiki → Migrate user passwords to Kitsune when logging in to Tiki
| Assignee | ||
Comment 1•15 years ago
|
||
Must have admin enabled in Django to test this. This checkout provides that:
http://github.com/pcraciunoiu/kitsune/commit/96585a787637ea022b4f37494482e4983e083ef3
Here's what I had to do to set up some other things:
* set up admin (bug 564101)
* touch wsgi/kitsune.wsgi
* add an alias to django's admin css, something like:
Alias /admin-media/ "/path/to/django/contrib/admin/media/"
That should be it, I think.
Attachment #446313 -
Flags: review?(james)
Comment 2•15 years ago
|
||
Comment on attachment 446313 [details] [diff] [review]
v1, admin rewrites + set django password on login (if changed) and on password change
>+# Rewrite admin URLs to Kitsune
>+RewriteRule ^admin(.*) /k/admin$1 [L,QSA]
>+RewriteRule ^([\-a-zA-Z]+)/admin(.*) /k/$1/admin$2 [L,QSA]
Going along with my comment on bug 564101, we really don't need to support locales here.
Otherwise, r+. Drop the locale in both places and it's working for me.
Attachment #446313 -
Flags: review?(james) → review+
| Assignee | ||
Comment 3•15 years ago
|
||
Woot! Dropped that one line in htaccess and committed - r67497 on trunk.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Updated•15 years ago
|
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•