webpass is not urlencoded

jhriggs - September 19, 2009 - 03:45
Project:User mailman register
Version:6.x-2.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

The admin password is not passed through urlencode() when passed as a query param. This caused my admin password (which happens to have a '+') to fail. The module always reported success for subscriptions, but I was never actually subscribed. Patch attached.

Additionally, you may want to check the resulting page for

<font color="#ff0000" size="+1"><strong><font size="+1">Authorization
failed.</font></strong>
to see if authorization failed. I would guess that mailman localizes this, though, so there may be something else you can check.

AttachmentSize
adminpw-urlencode.patch1.2 KB

#1

samuelet - September 23, 2009 - 13:04
Status:needs review» fixed

Patch applied in the dev release, thanks jhriggs.
The checking of server replies (not only the auth reply) is currently a big lack of UMR which i'm already working on.

#2

System Message - October 7, 2009 - 13:10
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.