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.

CommentFileSizeAuthor
adminpw-urlencode.patch1.2 KBjhriggs

Comments

samuelet’s picture

samuelet commented
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.

Status: Fixed » Closed (fixed)

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